Skip to content

security: Delay dependabot updates [TAROT-3707]#258

Merged
afsmeira merged 1 commit intomasterfrom
am/delay-dependabot-updates
May 4, 2026
Merged

security: Delay dependabot updates [TAROT-3707]#258
afsmeira merged 1 commit intomasterfrom
am/delay-dependabot-updates

Conversation

@afsmeira
Copy link
Copy Markdown
Contributor

7 days should be enough when most malicious packages are patched within 24 hours.

7 days should be enough when most malicious packages are patched within 24 hours.
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

Copy link
Copy Markdown

@codacy-production codacy-production Bot left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fails to achieve its primary security objective of delaying NuGet updates by 7 days. All reviewers identified that the cooldown and default-days properties used in .github/dependabot.yml are not supported by the official GitHub Dependabot configuration schema. Consequently, this configuration will either be ignored or cause a validation error, preventing the intended security delay from being applied. This is a critical blocker as it leaves the repository without the intended protection against immediate malicious package updates.

About this PR

  • The proposed configuration uses keys ('cooldown', 'default-days') that do not exist in the GitHub Dependabot v2 schema. Since Dependabot does not natively support delays based on package age, this PR will not function as intended.

Test suggestions

  • Validate .github/dependabot.yml against the official GitHub schema to ensure 'cooldown' is a supported property.
  • Verify that Dependabot successfully parses the configuration and waits 7 days after a package release before creating a PR.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Validate .github/dependabot.yml against the official GitHub schema to ensure 'cooldown' is a supported property.
2. Verify that Dependabot successfully parses the configuration and waits 7 days after a package release before creating a PR.

TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback

Comment thread .github/dependabot.yml
@afsmeira afsmeira changed the title security: Delay dependabot updates security: Delay dependabot updates [TAROT-3707] May 4, 2026
@afsmeira afsmeira merged commit 0132e09 into master May 4, 2026
5 checks passed
@afsmeira afsmeira deleted the am/delay-dependabot-updates branch May 4, 2026 13:28
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.

2 participants