Skip to content

fix(dependabot): set npm open-pull-requests-limit to 0#275

Open
don-petry wants to merge 32 commits into
mainfrom
claude/issue-261-compliance
Open

fix(dependabot): set npm open-pull-requests-limit to 0#275
don-petry wants to merge 32 commits into
mainfrom
claude/issue-261-compliance

Conversation

@don-petry
Copy link
Copy Markdown
Collaborator

Summary

  • Changes npm open-pull-requests-limit from 10 to 0 to implement the security-only policy per standards/dependabot-policy.md. Dependabot security updates bypass this limit, so security PRs still get created. Version-bump PRs (which don't affect application stability) are suppressed.

Compliance findings addressed

Finding Severity Status
wrong-limit-npm warning ✅ Fixed

Remaining findings requiring human action

  • Workflow SHA pinning (unpinned-actions-*) — requires workflows write permission
  • stray-codeql-workflow — delete codeql.yml and enable GitHub default setup via apply-repo-settings.sh
  • non-stub-dependabot-rebase.yml — sync from standards/workflows/dependabot-rebase.yml (requires workflows permission)
  • Repository settings (allow_auto_merge, delete_branch_on_merge, check-suite-auto-trigger-*) — run bash scripts/apply-repo-settings.sh google-app-scripts
  • Secret scanning settings — run bash scripts/apply-repo-settings.sh google-app-scripts

Part of petry-projects/.github#261

Generated with Claude Code

Copilot AI review requested due to automatic review settings May 12, 2026 19:13
@don-petry don-petry requested a review from a team as a code owner May 12, 2026 19:13
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 12, 2026

Warning

Rate limit exceeded

@github-actions[bot] has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 42 minutes and 47 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 51802e67-20a4-4483-b8d0-1d1846388fb4

📥 Commits

Reviewing files that changed from the base of the PR and between 98cc446 and e5946ac.

📒 Files selected for processing (1)
  • .github/dependabot.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-261-compliance

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

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.

Pull request overview

Updates the repository’s Dependabot configuration to follow the documented security-only policy for npm dependency updates by suppressing non-security version-bump PRs.

Changes:

  • Set npm open-pull-requests-limit from 10 to 0 in .github/dependabot.yml (security updates still bypass the limit).

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the .github/dependabot.yml configuration by setting the open-pull-requests-limit for the npm ecosystem to 0. The review feedback suggests applying this same change to the github-actions ecosystem to maintain consistency with the security-only policy across the repository.

Comment thread .github/dependabot.yml
schedule:
interval: 'weekly'
open-pull-requests-limit: 10
open-pull-requests-limit: 0
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The open-pull-requests-limit for npm is now 0, which aligns with the security-only policy mentioned in the pull request description. However, the github-actions ecosystem on line 16 still has this limit set to 10. To ensure this policy is applied consistently across all package ecosystems in this file, consider also setting the limit for github-actions to 0.

@don-petry
Copy link
Copy Markdown
Collaborator Author

@dev-lead - please fix this PR

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