Dependabot PRs should include cooldown settings in the PR descriptions #202727
Replies: 3 comments
|
💬 Your Product Feedback Has Been Submitted 🎉 Thank you for taking the time to share your insights with us! Your feedback is invaluable as we build a better GitHub experience for all our users. Here's what you can expect moving forward ⏩
Where to look to see what's shipping 👀
What you can do in the meantime 💻
As a member of the GitHub community, your participation is essential. While we can't promise that every suggestion will be implemented, we want to emphasize that your feedback is instrumental in guiding our decisions and priorities. Thank you once again for your contribution to making GitHub even better! We're grateful for your ongoing support and collaboration in shaping the future of our platform. ⭐ |
|
I like this idea. When I'm reviewing a Dependabot PR, I usually want more context than just "a new version is available." Knowing when that version was released can influence how comfortable I am merging it. For example, if a release is only a few hours old, I might prefer to let it sit for a few days in case regressions are discovered. If it's already been out for a couple of weeks, I'm generally more confident moving forward. Including something like:
would make that context immediately available without reviewers having to leave GitHub and look it up themselves. Even if it were just an optional section in the PR description, I think it would make reviewing Dependabot updates a bit smoother. |
|
Agreed — release age and the cooldown decision are exactly the review context that is missing today. A current workaround is to configure cooldown in each relevant version: 2
updates:
- package-ecosystem: npm
directory: /
schedule: { interval: daily }
cooldown:
default-days: 5
semver-major-days: 30
semver-minor-days: 7
semver-patch-days: 3The current cooldown reference says this applies to version updates, not security updates; the default is 3 days when no custom cooldown is set; configured values are 1–90 days; For the PR body, I would add one row per dependency (including grouped PRs):
The time calculation should be labeled derived, not presented as an internal scheduler verdict: package-manager metadata, grouped updates, and a later config edit can otherwise make the result look authoritative when it is only an audit snapshot. A small Action can provide that snapshot today by reading Dependabot metadata plus the registry/release timestamp and posting a check-run or comment. A native generated block plus machine-readable fields would make the context durable and refresh it when the PR is regenerated. |
Uh oh!
There was an error while loading. Please reload this page.
🏷️ Discussion Type
Product Feedback
💬 Feature/Topic Area
Other
Body
Dependabot cooldown is a useful option to help make dep updates secure. But at the moment a dependabot PR is being reviewed or approved, there is no context regarding the age of the dependency, or whether cooldown was in effect at the time the PR was introduced.
Dependabot PRs should include age information in the PR description (along with the current release information), and optionally include whether cooldown was in effect. This information is useful for conducting reviews.
All reactions