Skip to content

Commit e33984a

Browse files
committed
run spellchecker in separate job
1 parent 1ad9b30 commit e33984a

File tree

1 file changed

+16
-21
lines changed

1 file changed

+16
-21
lines changed

.github/workflows/ci.yml

Lines changed: 16 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,22 @@ jobs:
5858
COMMIT_MESSAGE: ${{ github.event.head_commit.message }}
5959
run: python3 src/ci/github-actions/calculate-job-matrix.py >> $GITHUB_OUTPUT
6060
id: jobs
61+
62+
spellcheck:
63+
name: run spellchecker
64+
runs-on: ubuntu-latest
65+
steps:
66+
- name: Checkout the source code
67+
uses: actions/checkout@v4
68+
69+
- name: check typos
70+
# sync version with src/tools/tidy/src/ext_tool_checks.rs
71+
uses: crate-ci/[email protected]
72+
with:
73+
# sync target files with src/tools/tidy/src/ext_tool_checks.rs
74+
files: ./compiler ./library
75+
config: ./typos.toml
76+
6177
job:
6278
name: ${{ matrix.name }}
6379
needs: [ calculate_matrix ]
@@ -104,18 +120,6 @@ jobs:
104120
with:
105121
fetch-depth: 2
106122

107-
- name: check typos
108-
id: typo_check
109-
if: ${{ env.CI_JOB_NAME == 'mingw-check-tidy' }}
110-
# don't fail job on typos
111-
continue-on-error: true
112-
# sync version with src/tools/tidy/src/ext_tool_checks.rs
113-
uses: crate-ci/[email protected]
114-
with:
115-
# sync target files with src/tools/tidy/src/ext_tool_checks.rs
116-
files: ./compiler ./library
117-
config: ./typos.toml
118-
119123
# Free up disk space on Linux by removing preinstalled components that
120124
# we do not need. We do this to enable some of the less resource
121125
# intensive jobs to run on free runners, which however also have
@@ -250,15 +254,6 @@ jobs:
250254
npm install -g @datadog/datadog-ci@^2.x.x
251255
python3 src/ci/scripts/upload-build-metrics.py build/cpu-usage.csv
252256
253-
# If pr have typos, we don't fail immediately, but allow job to continue,
254-
# but still want job to fail at the end. Without this, job will be marked as
255-
# 'success', even if there is typos, so fail it manually.
256-
#
257-
# also see https://github.com/actions/runner/issues/2347
258-
- name: check typos
259-
if: ${{ env.CI_JOB_NAME == 'mingw-check-tidy' && steps.typo_check.outcome == 'failure' && steps.typo_check.conclusion == 'success' }}
260-
run: exit 1
261-
262257
# This job isused to tell bors the final status of the build, as there is no practical way to detect
263258
# when a workflow is successful listening to webhooks only in our current bors implementation (homu).
264259
outcome:

0 commit comments

Comments
 (0)