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

Dependabot PRs use version information from incorrect repo #47

Merged
merged 1 commit into from
Mar 22, 2022
Merged

Conversation

miry
Copy link
Contributor

@miry miry commented Mar 22, 2022

The toxiproxy gem is fed by releases from the toxiproxy-ruby repo, but Dependabot gem bump PRs for this gem will link to and source the changelog from https://github.com/Shopify/toxiproxy/ instead, which is confusing and misleading. An example from a private repo: https://github.com/Shopify/pay/pull/14119

This may be caused by the gemspec file referring to https://github.com/Shopify/toxiproxy for most fields, including the homepage.

@miry
Copy link
Contributor

miry commented Mar 22, 2022

Checked the code of depndabot: https://github.com/dependabot/dependabot-core/blob/2f0db3e851ba2cc43d0b6dcd70da5e69d5b63eb6/bundler/lib/dependabot/bundler/metadata_finder.rb

Adding source_code_uri and changelog_uri

@miry miry self-assigned this Mar 22, 2022
@miry miry force-pushed the changelog branch 5 times, most recently from ed719b4 to 545a4e6 Compare March 22, 2022 09:43
@miry miry marked this pull request as ready for review March 22, 2022 09:44
@miry miry added the Toxiproxy label Mar 22, 2022
@miry miry force-pushed the changelog branch 2 times, most recently from 5d90eb5 to 598bbf7 Compare March 22, 2022 09:48
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Create CHANGELOG.md with 2 entries.
Update gemspec to have links to CHANGELOG.md.

Co-authored-by: Martin Contento <67421+tinnet@users.noreply.github.com>
@miry miry merged commit 69cb61e into master Mar 22, 2022
@miry miry deleted the changelog branch March 22, 2022 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants