Skip to content

metadata: trim .git suffix when searching github URL for gopkg #285

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

Merged
merged 1 commit into from
Jul 6, 2025

Conversation

n-peugnet
Copy link
Contributor

Some go hosts have this suffix in their go-import metadata which broke fetching the repo metadat from GitHub's API, e.g. go.bug.st:

Could not determine description for "go.bug.st/cleanup": GET https://api.github.com/repos/bugst/go-cleanup.git: 404 Not Found []
Could not determine long description for "go.bug.st/cleanup": get readme: GET https://api.github.com/repos/bugst/go-cleanup.git/readme: 404 Not Found []
Could not determine license for "go.bug.st/cleanup": get license for Go package: GET https://api.github.com/repos/bugst/go-cleanup.git/license: 404 Not Found []
Could not determine copyright for "go.bug.st/cleanup": get repo: GET https://api.github.com/repos/bugst/go-cleanup.git: 404 Not Found []

@n-peugnet n-peugnet changed the title metadata: trim .git suffix when searching gtihub URL for gopkg metadata: trim .git suffix when searching github URL for gopkg Jun 28, 2025
Some go hosts have this suffix in their go-import metadata which broke
fetching the repo metadat from GitHub's API, e.g. go.bug.st:

 Could not determine description for "go.bug.st/cleanup":
  GET https://api.github.com/repos/bugst/go-cleanup.git: 404 Not Found []
 Could not determine long description for "go.bug.st/cleanup": get readme:
  GET https://api.github.com/repos/bugst/go-cleanup.git/readme: 404 Not Found []
 Could not determine license for "go.bug.st/cleanup": get license for Go package:
  GET https://api.github.com/repos/bugst/go-cleanup.git/license: 404 Not Found []
 Could not determine copyright for "go.bug.st/cleanup": get repo:
  GET https://api.github.com/repos/bugst/go-cleanup.git: 404 Not Found []
@n-peugnet n-peugnet force-pushed the go-import-trim-suffix branch from 18db73d to 04e6e89 Compare June 28, 2025 10:55
Copy link
Contributor

@ottok ottok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@creekorful creekorful merged commit d47c5a8 into Debian:master Jul 6, 2025
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.

3 participants