Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: signoff commit #767

Merged
merged 4 commits into from
May 22, 2024
Merged

fix: signoff commit #767

merged 4 commits into from
May 22, 2024

Conversation

ramonpetgrave64
Copy link
Contributor

@ramonpetgrave64 ramonpetgrave64 commented May 7, 2024

Followup to #760

Fix the .github/workflows/update-actions-dist-post-commit.yml workflow to also signoff commit

Testing

Signed-off-by: Ramon Petgrave <32398091+ramonpetgrave64@users.noreply.github.com>
@ramonpetgrave64 ramonpetgrave64 changed the title signoff commit fix: signoff commit May 7, 2024
@ramonpetgrave64 ramonpetgrave64 enabled auto-merge (squash) May 7, 2024 16:33
@ramonpetgrave64
Copy link
Contributor Author

@ianlewis

@ramonpetgrave64 ramonpetgrave64 requested a review from ianlewis May 7, 2024 16:47
auto-merge was automatically disabled May 14, 2024 17:46

Pull request was closed

@ramonpetgrave64 ramonpetgrave64 enabled auto-merge (squash) May 14, 2024 17:47
@ramonpetgrave64 ramonpetgrave64 merged commit 18c5f13 into main May 22, 2024
17 checks passed
ramonpetgrave64 added a commit to ramonpetgrave64/slsa-verifier that referenced this pull request Jun 10, 2024
Followup to slsa-framework#760

Fix the .github/workflows/update-actions-dist-post-commit.yml workflow
to also signoff commit

# Testing

- [x] Invoked this PR's branch copy of the workflow against slsa-framework#717, and it
did signoff the commit.
-
slsa-framework@9670f76

Signed-off-by: Ramon Petgrave <32398091+ramonpetgrave64@users.noreply.github.com>
Signed-off-by: Ramon Petgrave <ramon.petgrave64@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants