Skip to content

Commit 67616d2

Browse files
committed
Merge branch 'alpha' into format_tools_evolving_5
2 parents ed72319 + ee5934a commit 67616d2

File tree

2 files changed

+27
-5
lines changed

2 files changed

+27
-5
lines changed

.github/workflows/standard_rust_push.yml

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,21 +64,32 @@ jobs :
6464
- name: Build module
6565
run: cd ${{ steps.rootpath.outputs.path }} && cargo build && cd -
6666
- name: Audit the modules
67+
id: audit
6768
run: make audit
6869
continue-on-error: true
6970
- name: Generate documentation for the modules
71+
id: documentation
7072
run: make doc open=no manifest_path=${{ inputs.manifest_path }}
7173
continue-on-error: true
7274
- name: Lint the modules
75+
id: lint
7376
run: make lint manifest_path=${{ inputs.manifest_path }} warnings=no
7477
continue-on-error: true
7578
- name: Check the modules
79+
id: check
7680
run: make check manifest_path=${{ inputs.manifest_path }}
7781
continue-on-error: true
7882
- name: Check the modules dependencies
83+
id: udeps
7984
run: cargo +nightly udeps --all-targets --manifest-path ${{ inputs.manifest_path }}
80-
# This part means that all previous steps will be executed and workflow will not stop if they fail, but not this one.
81-
continue-on-error: false
85+
continue-on-error: true
86+
# Added IDs for each step in the test job: This allows referencing the result of each step later.
87+
#
88+
# "Check for errors" step: Now checks the outcome status for each step.
89+
# If any of them have a value of 'failure', this step will fail the job by returning exit 1.
90+
- name: Check for errors
91+
if: steps.audit.outcome != 'success' || steps.documentation.outcome != 'success' || steps.lint.outcome != 'success' || steps.check.outcome != 'success' || steps.udeps.outcome != 'success'
92+
run: exit 1
8293

8394
# release:
8495
# if: contains( inputs.commit_message, '+test' ) || contains( inputs.commit_message, 'merge' )
@@ -127,7 +138,7 @@ jobs :
127138

128139
will_test :
129140
# This section ensures that `job` `will_test` will only be executed after `checkmate` and if `checkmate` fails, no tests will be run.
130-
needs:
141+
needs :
131142
- checkmate
132143
if : contains( inputs.commit_message, '+test' ) || inputs.commiter_username == 'web-flow' || startsWith( inputs.commit_message, 'merge' )
133144
concurrency :

module/move/willbe/template/workflow/standard_rust_push.yml

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,21 +64,32 @@ jobs :
6464
- name: Build module
6565
run: cd ${{ steps.rootpath.outputs.path }} && cargo build && cd -
6666
- name: Audit the modules
67+
id: audit
6768
run: make audit
6869
continue-on-error: true
6970
- name: Generate documentation for the modules
71+
id: documentation
7072
run: make doc open=no manifest_path=${{ inputs.manifest_path }}
7173
continue-on-error: true
7274
- name: Lint the modules
75+
id: lint
7376
run: make lint manifest_path=${{ inputs.manifest_path }} warnings=no
7477
continue-on-error: true
7578
- name: Check the modules
79+
id: check
7680
run: make check manifest_path=${{ inputs.manifest_path }}
7781
continue-on-error: true
7882
- name: Check the modules dependencies
83+
id: udeps
7984
run: cargo +nightly udeps --all-targets --manifest-path ${{ inputs.manifest_path }}
80-
# This part means that all previous steps will be executed and workflow will not stop if they fail, but not this one.
81-
continue-on-error: false
85+
continue-on-error: true
86+
# Added IDs for each step in the test job: This allows referencing the result of each step later.
87+
#
88+
# "Check for errors" step: Now checks the outcome status for each step.
89+
# If any of them have a value of 'failure', this step will fail the job by returning exit 1.
90+
- name: Check for errors
91+
if: steps.audit.outcome != 'success' || steps.documentation.outcome != 'success' || steps.lint.outcome != 'success' || steps.check.outcome != 'success' || steps.udeps.outcome != 'success'
92+
run: exit 1
8293

8394
# release:
8495
# if: contains( inputs.commit_message, '+test' ) || contains( inputs.commit_message, 'merge' )

0 commit comments

Comments
 (0)