Skip to content

Track and commit only files modified by digest updates#62

Merged
cpanato merged 1 commit intochainguard-dev:mainfrom
navikt:main
Nov 12, 2025
Merged

Track and commit only files modified by digest updates#62
cpanato merged 1 commit intochainguard-dev:mainfrom
navikt:main

Conversation

@albrektsson
Copy link
Contributor

@albrektsson albrektsson commented Sep 24, 2025

  • Add changed_files array to track files actually modified during digest updates
  • Output list of changed files as new action output parameter
  • Use changed_files list in create-pull-request add-paths to avoid committing unrelated file changes
  • Add test for only-changed-digests-committed scenario
  • Normalize filepaths to fix mismatch issues
  • Update README with new functionality

I'm not saying I accidentally pushed credentials to github, but if I did this change would've prevented it. 🙄

@albrektsson
Copy link
Contributor Author

Oops, forgot to detach fork 😢

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

some comments

@cpanato
Copy link
Member

cpanato commented Oct 1, 2025

also please squash your commits

- Add changed_files array to track files actually modified during digest updates
- Output list of changed files as new action output parameter
- Use changed_files list in create-pull-request add-paths to avoid committing unrelated file changes
- Add test for only-changed-digests-committed scenario
- Normalize filepaths to fix mismatch issues
- Update README with new functionality
@albrektsson
Copy link
Contributor Author

There we go, sorry for the mess :D

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

seems good
thanks

@cpanato cpanato merged commit f6b54e7 into chainguard-dev:main Nov 12, 2025
7 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.

2 participants