Skip to content

security: Delay dependabot updates#339

Open
afsmeira wants to merge 1 commit intomasterfrom
am/delay-dependabot-updates
Open

security: Delay dependabot updates#339
afsmeira wants to merge 1 commit intomasterfrom
am/delay-dependabot-updates

Conversation

@afsmeira
Copy link
Copy Markdown

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 cannot be merged in its current state as it introduces an invalid configuration key, cooldown, which is not supported by the GitHub Dependabot schema. Adding unrecognized keys will cause Dependabot to fail to parse the file, potentially disabling dependency updates entirely for this repository.

While the goal of delaying updates for security reasons is valid, Dependabot does not natively support a version-age or cooldown feature. To achieve the intended 7-day delay, you would need to use a different tool like Renovate with stabilityDays or reduce the update frequency to weekly to minimize the noise of immediate releases.

About this PR

  • The proposed configuration change will not achieve the desired outcome because GitHub Dependabot lacks a native feature to delay updates based on release age. Implementing this would require external workflow logic or switching to a dependency manager with built-in stability delays.

Test suggestions

  • Verify that Dependabot correctly parses the configuration without schema errors.
  • Confirm that updates are only proposed for versions that have been released for at least 7 days.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that Dependabot correctly parses the configuration without schema errors.
2. Confirm that updates are only proposed for versions that have been released for at least 7 days.

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

Comment thread .github/dependabot.yml
Comment on lines +9 to +10
cooldown:
default-days: 7
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔴 HIGH RISK

The cooldown property is not a valid GitHub Dependabot configuration option. Including unsupported keys will cause a schema validation error, which may lead Dependabot to ignore the configuration file entirely or report a syntax error in the repository's Security tab. If the intent is to delay updates until they reach a certain age to avoid malicious or buggy releases, Dependabot does not currently support this feature. Consider changing the interval to weekly to reduce frequency, or evaluate using Renovate which supports a stabilityDays option.

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.

1 participant