Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix go dependencies #99

Merged
merged 3 commits into from
Jan 3, 2024
Merged

Fix go dependencies #99

merged 3 commits into from
Jan 3, 2024

Conversation

danielrbradley
Copy link
Member

@danielrbradley danielrbradley commented Jan 3, 2024

Fixes #98

  1. Upgrade dependencies
  2. Replace sourcegraph.com/sourcegraph/appdash-data
  3. Use older version in replacement to avoid go panic when module version already referenced:

panic: unexpected output encountered: go: github.com/sourcegraph/[email protected] used for two different module paths (github.com/sourcegraph/appdash-data and sourcegraph.com/sourcegraph/appdash-data) (Did you use -json on go test?)

Sample error when replacing to an already referenced path:

panic: unexpected output encountered: go: github.com/sourcegraph/[email protected] used for two different module paths (github.com/sourcegraph/appdash-data and sourcegraph.com/sourcegraph/appdash-data) (Did you use -json on go test?)
@danielrbradley danielrbradley requested a review from a team January 3, 2024 13:35
@danielrbradley danielrbradley merged commit 7537b91 into main Jan 3, 2024
5 checks passed
@danielrbradley danielrbradley deleted the fix-go-deps branch January 3, 2024 13:39
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.

Workflow failure: main
2 participants