-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* ci: trigger separate workflow * ci: temporary use current branch * ci: fix workflow name * ci: try with same job name * ci: try with dispatch * Revert "ci: try with dispatch" This reverts commit bd66e56. * Revert "ci: try with same job name" This reverts commit 9e2ae5b. * ci: try with workflow call in both cases * ci: introduce change to trigger CI * Revert "ci: introduce change to trigger CI" This reverts commit e3ec07c. * ci: add name * Revert "Revert "ci: introduce change to trigger CI"" This reverts commit 6718585. * ci: improve naming * ci: further improve naming * Unset reusable workflow version and use relative path * Remove CI trigger --------- Co-authored-by: Silvano Cerza <silvanocerza@gmail.com>
- Loading branch information
1 parent
40360e4
commit 451fae8
Showing
3 changed files
with
34 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
# If you change this name also do it in tests.yml and ci_metrics.yml | ||
# We use a separate workflow to skip the tests if the PR contains both code and non-code changes (e.g. release notes). | ||
# Skipping the job unfortunately doesn't work because GitHub will treat these jobs as successful even if they are | ||
# skipped. Hence, we need to revert to a separate workflow. | ||
name: Tests | ||
on: | ||
workflow_call: | ||
inputs: | ||
tests_were_skipped: | ||
type: boolean | ||
required: true | ||
|
||
jobs: | ||
catch-all: | ||
# Don't run this check if the PR contains both code and non-code changes (e.g. release notes) | ||
name: Mark tests as completed | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Skip tests | ||
if: ${{ github.event.inputs.tests_were_skipped }} | ||
run: echo "Skipped!" | ||
- name: Tests completed successfully | ||
if: ${{ !github.event.inputs.tests_were_skipped }} | ||
run: echo "Tests completed!" |