Skip to content

ci: hold cdk-nag and @types/node major bumps in Dependabot#276

Merged
frbrkoala merged 1 commit into
mainfrom
chore/dependabot-hold-major-bumps
Jul 6, 2026
Merged

ci: hold cdk-nag and @types/node major bumps in Dependabot#276
frbrkoala merged 1 commit into
mainfrom
chore/dependabot-hold-major-bumps

Conversation

@frbrkoala

Copy link
Copy Markdown
Contributor

Add ignore rules to the root npm ecosystem so Dependabot stops proposing major version bumps for two dependencies (minor/patch updates still flow):

  • cdk-nag: v3 is a breaking rewrite (IAspect -> IPolicyValidationPlugin, and NagSuppressions -> per-finding Validations.acknowledge). Deferred until upstream supports prefix/bulk acknowledgment. This is why the grouped npm PR chore: bump the npm group across 1 directory with 4 updates #275 fails "Validate CDK synthesis" — the ignore drops the v3 bump so a regenerated PR builds. Unignore when ready to do the v3 migration.
  • @types/node: keep aligned with the Node.js runtime major we target (currently 24) rather than chasing the newest typings.

What does this PR do?

🛑 Please open an issue first to discuss any significant work and flesh out details/direction — we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • New blueprint (adding a new protocol — see checklist below)
  • Documentation update
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Testing

  • I have run npm run build successfully
  • I have run npm run test and all tests pass
  • I have run npx cdk synth with a sample .env (e.g. from blueprints/dummy/samples/) and it succeeds
  • I have run pre-commit hooks: npm run run-pre-commit

New Blueprint Checklist (if adding a protocol)

  • Blueprint follows the structure of blueprints/dummy/ (reference implementation)
  • Blueprint README follows the standard template (matches Dummy section titles and ordering)
  • Setup section points to Getting Started (not inline instructions)
  • Deployment section leads with AI-driven deployment (Option 1) and manual as Option 2
  • Sample .env files are provided for at least one network (mainnet or testnet)
  • Configuration scripts follow the install/run dispatch pattern (see blueprints/dummy/)
  • Blueprint page added to website/docs/blueprints/ (.mdx mirror importing README)
  • Client Release Channels table added to README under Additional Resources

Documentation

  • I have updated relevant documentation (if applicable)
  • I have run cd website && npm run build with no broken links (if docs changed)

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add ignore rules to the root npm ecosystem so Dependabot stops proposing major
version bumps for two dependencies (minor/patch updates still flow):

- cdk-nag: v3 is a breaking rewrite (IAspect -> IPolicyValidationPlugin, and
  NagSuppressions -> per-finding Validations.acknowledge). Deferred until
  upstream supports prefix/bulk acknowledgment. This is why the grouped npm PR
  #275 fails "Validate CDK synthesis" — the ignore drops the v3 bump so a
  regenerated PR builds. Unignore when ready to do the v3 migration.
- @types/node: keep aligned with the Node.js runtime major we target
  (currently 24) rather than chasing the newest typings.
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Scan for commit: 457a57a | Updated: 2026-07-06 05:04:10 UTC

🔒 Security Scan Results

Scanned files: 1

⚠️ Security findings detected. Please review and address before merging.

@frbrkoala frbrkoala merged commit a304ea4 into main Jul 6, 2026
7 checks passed
@frbrkoala frbrkoala deleted the chore/dependabot-hold-major-bumps branch July 6, 2026 05:04
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