Skip to content

Commit

Permalink
feat: run fmt and spell checks first (#569)
Browse files Browse the repository at this point in the history
* feat(ci): spell and format first

* chore: debug

* chore: simic

* fix: check needs build

* chore(ci): remove spelling from test pashe

* chore: revert "chore: debug"

This reverts commit e253daf.

* chore: remove filter.jq
  • Loading branch information
PoisonPhang authored Aug 29, 2023
1 parent 057ebce commit 551760f
Showing 1 changed file with 33 additions and 2 deletions.
35 changes: 33 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,33 @@ concurrency:
cancel-in-progress: true

jobs:
spell-fmt-check:
uses: unionlabs/workflows/.github/workflows/build.yml@2c94f6f3acb2d7d9222f8fa05f43eb8c3ed2e758
secrets:
nixbuild_token: ${{ secrets.nixbuild_token }}
access-tokens: github.com=${{ secrets.GITHUB_TOKEN }}
with:
filter_builds: |
(
(.top_attr == "checks") and
(.system == "x86_64-linux") and
(
(
# Do not run on both systems
[.attr] | inside(
[
"spellcheck",
"treefmt-check"
]
)
)
)
)
build:
needs: [ spell-fmt-check ]
if: |
always()
&& (contains(needs.*.result, 'success') || github.event.pull_request.draft == true)
uses: unionlabs/workflows/.github/workflows/build.yml@2c94f6f3acb2d7d9222f8fa05f43eb8c3ed2e758
secrets:
nixbuild_token: ${{ secrets.nixbuild_token }}
Expand Down Expand Up @@ -67,8 +93,11 @@ jobs:
)
)
)
check:
test:
needs: [ build ]
if: |
always()
&& contains(needs.*.result, 'success')
uses: unionlabs/workflows/.github/workflows/build.yml@2c94f6f3acb2d7d9222f8fa05f43eb8c3ed2e758
secrets:
nixbuild_token: ${{ secrets.nixbuild_token }}
Expand All @@ -80,7 +109,7 @@ jobs:
(.system == "x86_64-linux") and
(
(
# Do not run on both systems
# Do not run
[.attr] | inside(
[
"ensure-blocks",
Expand All @@ -89,6 +118,8 @@ jobs:
"pre-commit",
"pre-commit-check",
"sepolia-runs",
"spellcheck",
"treefmt-check",
"union-runs",
"virtualisation-works"
]
Expand Down

0 comments on commit 551760f

Please sign in to comment.