Skip to content

Switch go-snipeit from CampusTech fork to upstream#31

Merged
robbiet480 merged 2 commits into
mainfrom
refactor/upstream-go-snipeit
Mar 8, 2026
Merged

Switch go-snipeit from CampusTech fork to upstream#31
robbiet480 merged 2 commits into
mainfrom
refactor/upstream-go-snipeit

Conversation

@robbiet480

@robbiet480 robbiet480 commented Mar 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Switch go-snipeit dependency from CampusTech/go-snipeit fork back to upstream michellepellon/go-snipeit
  • All our improvements were merged upstream via michellepellon/go-snipeit#3
  • Updates all import paths across snipe/, sync/, and test files

Test plan

  • go build ./... passes
  • go test ./... passes (all 4 test suites)
  • Run ./axm2snipe sync --dry-run -v to verify end-to-end

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated project dependencies and import references across the codebase to use updated library sources.

PR #3 (michellepellon/go-snipeit) merged all our improvements upstream.
Switch back to the canonical module path.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@robbiet480 robbiet480 enabled auto-merge March 8, 2026 05:09
@coderabbitai

coderabbitai Bot commented Mar 8, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3b518830-a852-4045-b1fa-b9f422334382

📥 Commits

Reviewing files that changed from the base of the PR and between 7a1d882 and 228ef7e.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (5)
  • go.mod
  • snipe/client.go
  • snipe/client_test.go
  • sync/sync.go
  • sync/sync_test.go

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


Walkthrough

The pull request migrates the go-snipeit dependency source from the CampusTech fork to the michellepellon fork. All import paths are updated across the go.mod file and four Go source files (two in snipe/ and two in sync/) to reflect this change, with no functional logic modifications.

Changes

Cohort / File(s) Summary
Dependency Update
go.mod
Updated go-snipeit module requirement from github.com/CampusTech/go-snipeit to github.com/michellepellon/go-snipeit with corresponding version change.
Import Path Updates
snipe/client.go, snipe/client_test.go, sync/sync.go, sync/sync_test.go
Updated import statements across all files to reference the new michellepellon fork instead of CampusTech fork. No logic or behavior changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

Poem

🐰 A fork in the road, we've chosen anew,
From CampusTech's branch to michellepellon's view,
Four files aligned, imports all set,
The snipeit dependency—refreshed, no regret! ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/upstream-go-snipeit

Comment @coderabbitai help to get the list of available commands and usage tips.

@robbiet480

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Mar 8, 2026

Copy link
Copy Markdown
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@robbiet480 robbiet480 merged commit be92957 into main Mar 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant