Skip to content
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

workflows: enable commit signing for BrewTestBot #105988

Merged
merged 1 commit into from
May 19, 2021
Merged

Conversation

Rylan12
Copy link
Member

@Rylan12 Rylan12 commented May 18, 2021

This PR adds the setup-commit-signing action to enable commit signing for BrewTestBot, wherever required. See Homebrew/brew#11403.

CC @nandahkrishna

@Rylan12 Rylan12 added the ci-syntax-only Only run syntax checks on CI. Use only for bulk changes. label May 18, 2021
@Rylan12
Copy link
Member Author

Rylan12 commented May 18, 2021

@nandahkrishna is there an easy way for us to test this out or should we just merge and assume it will work (I'll try to keep an eye out for failed workflows or unsigned commits in PRs)

@nandahkrishna
Copy link
Member

nandahkrishna commented May 18, 2021

Do we have a way to manually run any of these workflows? We could try creating a test PR (perhaps with bump-unversioned-casks) and see if the commits are signed. Don't think there's an easier way to test it 😅

@Rylan12
Copy link
Member Author

Rylan12 commented May 18, 2021

So the bump-unversioned-casks workflow can be run manually but I didn't realize that it's been manually disabled (looks like for around a month).

I think I'd feel comfortable merging without testing because this is something that works in lots of other places

@miccal miccal merged commit ae2f658 into master May 19, 2021
@miccal miccal deleted the commit-signing branch May 19, 2021 03:14
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge-skip ci-syntax-only Only run syntax checks on CI. Use only for bulk changes. outdated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants