-
Notifications
You must be signed in to change notification settings - Fork 7.9k
CI: Cirrus CI - skip build on ignore-paths #13675
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
e6e47d0
to
1a665a0
Compare
31bb68c
to
bc16233
Compare
Rebased after the openssl min version bump and FreeBSD image bump. |
bc16233
to
b8c8359
Compare
b8c8359
to
696ece0
Compare
696ece0
to
715a7d9
Compare
Rebased and pushed, including some changes now that we no longer use Travis CI. Cirrus CI doc on the Requesting @TimWolla if you could take a look too please? Thank you. |
Cirrus CI supports a `<job>.skip` key that skips the build if the expression evaluates to true. This adds the same list of `on.pull_request.paths-ignore` patterns from GitHub Actions workflows. This should save several minutes of CI times on PRs when the changes are only in the README/doc files. [Cirrus CI Docs](https://cirrus-ci.org/guide/writing-tasks/#supported-functions)
715a7d9
to
131fe02
Compare
This has apparently forgotten. Sorry. Thank you @Ayesh! |
Sorry, I may have assumed you have commit karma. ^^ |
May be time for php.net account, and commit karma for php-src? See https://www.php.net/git-php.php. |
Thank you so much for your kind words! I submitted the form and have my fingers crossed :) |
Cirrus CI supports a
<job>.skip
key that skips the build if the expression evaluates to true. This adds the same list of on.pull_request.paths-ignore patterns from GitHub Actions workflows.This should save several minutes of CI times on PRs when the changes are only in the README/doc files.