Skip to content

fix: allow Dependabot action pin updates in validation - #39

Merged
cshiels-ie merged 1 commit into
ansible:develfrom
cshiels-ie:fix/validate-action-pin-changes
Sep 10, 2026
Merged

cshiels-ie merged 1 commit into
ansible:develfrom
cshiels-ie:fix/validate-action-pin-changes

Conversation

@cshiels-ie

Copy link
Copy Markdown
Contributor

Summary

  • Ignore GitHub Action SHA-only differences when validating protected workflow files against the recorded framework template.
  • Continue reporting substantive workflow changes and protected-file modifications.
  • Add a regression test covering Dependabot-style action pin changes.

This unblocks metrics-service PR #419 and similar downstream Dependabot updates without requiring every downstream project to update its .copier-answers.yml framework commit.

Testing

  • uv run ruff check src/platform_service_framework/cli.py
  • git diff --check
  • Clean-checkout validation test to be run by CI.

AI Assistance

This change was developed with assistance from OpenCode (openai/gpt-5.6-luna).
All generated code was reviewed, tested, and validated before merging.

Allow downstream Dependabot updates to framework-managed GitHub Action SHAs while continuing to reject substantive protected workflow changes.

Assisted-by: OpenCode <openai/gpt-5.6-luna>
@cshiels-ie
cshiels-ie marked this pull request as ready for review September 10, 2026 20:37
@cshiels-ie
cshiels-ie merged commit 219774c into ansible:devel Sep 10, 2026
7 checks passed
@cshiels-ie
cshiels-ie deleted the fix/validate-action-pin-changes branch September 10, 2026 20:38
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