Skip to content

build(workflows): fix stale version comment for dependency-review-action v4.9.0#948

Merged
WilliamBerryiii merged 2 commits intomainfrom
feat/issue-891-dependency-review-uv-lock
Mar 9, 2026
Merged

build(workflows): fix stale version comment for dependency-review-action v4.9.0#948
WilliamBerryiii merged 2 commits intomainfrom
feat/issue-891-dependency-review-uv-lock

Conversation

@WilliamBerryiii
Copy link
Copy Markdown
Member

@WilliamBerryiii WilliamBerryiii commented Mar 9, 2026

Description

Fix stale version comment in the dependency-review workflow. Dependabot PR #942 updated the dependency-review-action SHA to v4.9.0, but left the trailing version comment at # v4.3.4. This PR corrects the comment to # v4.9.0 so it accurately reflects the pinned action version.

Single comment-only change — no functional or behavioral impact.

Related Issue(s)

Closes #949

Type of Change

Select all that apply:

Code & Documentation:

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update

Infrastructure & Configuration:

  • GitHub Actions workflow
  • Linting configuration (markdown, PowerShell, etc.)
  • Security configuration
  • DevContainer configuration
  • Dependency update

AI Artifacts:

  • Reviewed contribution with prompt-builder agent and addressed all feedback
  • Copilot instructions (.github/instructions/*.instructions.md)
  • Copilot prompt (.github/prompts/*.prompt.md)
  • Copilot agent (.github/agents/*.agent.md)
  • Copilot skill (.github/skills/*/SKILL.md)

Other:

  • Script/automation (.ps1, .sh, .py)
  • Other (please describe):

Testing

Automated validation (all passed):

Command Result
npm run lint:version-consistency Pass
npm run lint:dependency-pinning Pass
npm run lint:all Pass

Security analysis: No concerns — change is comment-only with no dependency additions, permission changes, or sensitive data exposure.

Manual testing: Not performed — comment-only change has no runtime behavior to test.

Checklist

Required Checks

  • Documentation is updated (if applicable) — (N/A — no documentation impact)
  • Files follow existing naming conventions
  • Changes are backwards compatible (if applicable) — (N/A — comment-only change)
  • Tests added for new functionality (if applicable) — (N/A — no new functionality)

AI Artifact Contributions

Required Automated Checks

The following validation commands must pass before merging:

  • Markdown linting: npm run lint:md
  • Spell checking: npm run spell-check
  • Frontmatter validation: npm run lint:frontmatter
  • Skill structure validation: npm run validate:skills
  • Link validation: npm run lint:md-links
  • PowerShell analysis: npm run lint:ps
  • Plugin freshness: npm run plugin:generate

Security Considerations

  • This PR does not contain any sensitive or NDA information
  • Any new dependencies have been reviewed for security issues — (N/A — no dependency changes, comment fix only)
  • Security-related scripts follow the principle of least privilege — (N/A — no security scripts modified)

Additional Notes

The SHA 2031cfc080254a8a887f58cffee85186f0e49e48 was already correct for v4.9.0 (landed via dependabot PR #942). Only the trailing version comment was stale.

…ion v4.9.0

- correct version comment from v4.3.4 to v4.9.0 for dependency-review-action
- SHA 2031cfc was already updated to v4.9.0 via dependabot PR #942
- addresses #891

🔒 - Generated by Copilot
@WilliamBerryiii WilliamBerryiii requested a review from a team as a code owner March 9, 2026 00:02
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 9, 2026

Dependency Review

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

Scanned Files

None

@WilliamBerryiii WilliamBerryiii merged commit 45c9262 into main Mar 9, 2026
25 checks passed
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.

build(workflows): fix stale version comment for dependency-review-action v4.9.0

2 participants