Skip to content

Conversation

charlespierce
Copy link
Collaborator

Info

  • Second attempt at an automated job to update flake.lock, after Add automated job to update flake.lock file #264
  • The original job worked, but the PR created did not have signed commits, so it wasn't mergeable.
  • This approach adds a GPG key and associates the commits with @1password-bot (which is already used in other repos within the org)

Changes

  • The job is created the same way as the previous PR, however we now add the author, committer, and GPG key information using repo secrets.

Testing

  • We discovered in the last PR that testing via a PR-triggered job did not work as expected, so we'll likely need some quick follow-up testing once this is merged.

Copy link
Collaborator

@LuminaSapphira LuminaSapphira left a comment

Choose a reason for hiding this comment

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

Hopefully this one works!

@charlespierce charlespierce merged commit f48493d into main Aug 28, 2025
6 checks passed
@charlespierce charlespierce deleted the cap/automated_flake_updates_2 branch August 28, 2025 16:59
@charlespierce
Copy link
Collaborator Author

Success! #268 (and even merged)

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