Skip to content

Update Dependabot to ignore patch updates#271

Merged
jbangelo merged 4 commits intomasterfrom
jbangelo/skip-patch-versions
Mar 6, 2025
Merged

Update Dependabot to ignore patch updates#271
jbangelo merged 4 commits intomasterfrom
jbangelo/skip-patch-versions

Conversation

@jbangelo
Copy link
Contributor

@jbangelo jbangelo commented Mar 3, 2025

We tend to get a bunch of patch updates to dependencies , which can create a lot of pointless noise in the commit history. Since we have security scans enabled, it should be fine to skip over patch updates until we see encounter a bug that requires an update.

This also adds a reviewer to the Dependabot PRs, so that we can get them reviewed and merged in a timely fashion.

@jbangelo jbangelo requested a review from Copilot March 3, 2025 18:26
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This PR updates the Dependabot configuration to reduce noise by ignoring patch updates for dependencies and to streamline reviews by automatically assigning a reviewer.

  • Adds a reviewers list with "jbangelo"
  • Introduces a general ignore rule for patch updates across all dependencies
  • Retains specific pinned version rules for vergen and skips a specific version of curl

Reviewed Changes

File Description
.github/dependabot.yml Updated configuration to reduce patch update noise and assign reviewers

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Copy link

@john-michaelburke john-michaelburke left a comment

Choose a reason for hiding this comment

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

you could probably throw a group on there like the quality group or something unless you'd prefer to be blasted on your own. Seems like we could benefit from like a generic rust reviewer group which you could use for things like this

Copy link
Contributor

@pcrumley pcrumley left a comment

Choose a reason for hiding this comment

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

I hear where you are coming from but this is a little heavy handed in your approach.

at the very least dependabot knows security updates vs patch and we should allow those. I also think it is nice to group updates to avoid spam. I found this setting works well for my rust repos:

---
version: 2
updates:
  - package-ecosystem: "cargo"
    directory: "/"
    schedule:
      interval: "daily"
    groups:
      minor-and-patch:
        applies-to: version-updates
        update-types:
          - patch
          - minor
      security-updates:
        applies-to: security-updates
        update-types:
          - patch
          - minor
  - package-ecosystem: github-actions
    directory: "/"
    schedule:
      interval: weekly

- 0.4.36
# Ignore patch updates for all
- dependency-name: "*"
update-types: ["version-update:semver-patch"]
Copy link
Contributor

Choose a reason for hiding this comment

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

actually this already only affects version-updates. sorry for being trigger happy with changes requested. I think it makes sense to group instead of disabling if you are worried about git log spam but this is fine as is if you are happy with it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe doing less often checks in addition to grouping them together would be the right path forward?

Copy link
Contributor

@pcrumley pcrumley Mar 3, 2025

Choose a reason for hiding this comment

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

@jbangelo that's what I would recommend, but i don't feel strongly about it. You don't have to merge the PRs in right away and dependabot will just add new ones to the group, so i don't find the daily checks too annoying, i just merge them in like once every couple weeks

@pcrumley
Copy link
Contributor

pcrumley commented Mar 3, 2025

fyi there isn't really a good reason to keep vergen pinned at version three, other than the breaking change is confusing to fix. Luckily i did that for other rust tools awhile back, and I updated ntripping to the latest vergen version in line with other rust tools we have: #272

@jbangelo
Copy link
Contributor Author

jbangelo commented Mar 3, 2025

you could probably throw a group on there like the quality group or something unless you'd prefer to be blasted on your own. Seems like we could benefit from like a generic rust reviewer group which you could use for things like this

Changed out me for the @swift-nav/open-source group, just had to make sure we had added that team to this repo.

@jbangelo jbangelo merged commit 1b79afe into master Mar 6, 2025
6 checks passed
@jbangelo jbangelo deleted the jbangelo/skip-patch-versions branch March 6, 2025 02:38
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.

4 participants