Skip to content

Commit Verification 🔒 #333

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Dec 9, 2024
Merged

Commit Verification 🔒 #333

merged 9 commits into from
Dec 9, 2024

Conversation

GrantBirki
Copy link
Member

@GrantBirki GrantBirki commented Dec 9, 2024

This pull request introduces several changes to add commit verification functionality and update related tests and documentation.

This PR adds the new commit_verification input option which can be used to enforce commit verification before a deployment can continue. If commit_verification: "true" is set ("false" by default), the deployment will only proceed if the commit is verified and the verified_at date occurs before the .deploy comment was issued.

A new commit_verified output variable has also been added which reflects the outcome of the commit verification check.

Lastly, you can use the commit_verified template variable when rendering custom deployment messages.

@GrantBirki GrantBirki added the enhancement New feature or request label Dec 9, 2024
@GrantBirki GrantBirki self-assigned this Dec 9, 2024

This comment was marked as outdated.

…ecks()` and ensure `commitSafetyChecks()` is bypassed during stable branch deployments

This comment was marked as outdated.

@GrantBirki GrantBirki marked this pull request as ready for review December 9, 2024 21:54
@GrantBirki GrantBirki merged commit 525c08a into main Dec 9, 2024
4 checks passed
@GrantBirki GrantBirki deleted the commit-verification branch December 9, 2024 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants