Skip to content

Conversation

balos1
Copy link
Member

@balos1 balos1 commented Oct 7, 2025

This will limit the GitHub actions CI to only commits with "[run-ci]" in the commit message. This will ensure we are not running the GitHub actions CI more than necessary.

@balos1
Copy link
Member Author

balos1 commented Oct 7, 2025

With this requirement, we could also allow CI to run on pushes to branches, provided the [run-ci] tag is in the commit message.

@balos1 balos1 changed the title run ci only if [run-ci] in commit message run ci only if [run-ci] is in commit message Oct 7, 2025
@balos1 balos1 added the ci label Oct 7, 2025
drreynolds
drreynolds previously approved these changes Oct 8, 2025
Copy link
Collaborator

@drreynolds drreynolds left a comment

Choose a reason for hiding this comment

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

This looks fine, but can you clarify the purpose of the PR? Is it just that we run the CI too frequently and want to limit this? Or is this a "security" question?

@balos1
Copy link
Member Author

balos1 commented Oct 9, 2025

We run CI too frequently and want to limit it, especially if we start hosting our own github runners. We can discuss this more at our next team meeting. For now, we should not merge this. I meant to open it as a draft.

@balos1
Copy link
Member Author

balos1 commented Oct 9, 2025

Ill also note that GitHub supports a skip ci command by default: https://docs.github.com/en/actions/how-tos/manage-workflow-runs/skip-workflow-runs. We could use an opt-out approach instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants