Skip to content

Commit

Permalink
chore: improve CI parallelism (#263)
Browse files Browse the repository at this point in the history
Moving the Subverso consistency check to its own job improves parallel
processing and fails faster.

Also, there's no need to run on push and on pull request - it'd be
nice to kick off builds when the push happens, but this results in
double builds, which is not good.
  • Loading branch information
david-christiansen authored Jan 6, 2025
1 parent fb294c7 commit b84e7f6
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 6 deletions.
8 changes: 2 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
on:
push:
branches:
- master
pull_request:

name: Continuous Integration
Expand Down Expand Up @@ -59,12 +61,6 @@ jobs:
jobSummary: true
args: --base './_out/examples/demosite' --no-progress --offline './_out/examples/demosite/**/*.html'

- name: Check for consistent Subverso versions in all manifests
run: |
echo "Subverso versions:"
find . -name lake-manifest.json -print0 | xargs -0 jq '.packages[] | select(.name == "subverso") | {"file": input_filename, "subverso": .rev}'
find . -name lake-manifest.json -print0 | xargs -0 jq -e -s 'map(.packages[] | select(.name == "subverso").rev) | .[0] as $x | all(.[]; . == $x)'
- name: Install PDF Dependencies
uses: teatimeguest/setup-texlive-action@v3
with:
Expand Down
18 changes: 18 additions & 0 deletions .github/workflows/consistent-subverso-manifests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
on:
push:
branches:
- master
pull_request:

name: Consistent Subverso dependencies

jobs:
build:
name: consistency
runs-on: ubuntu-latest
steps:
- name: Check for consistent Subverso versions in all manifests
run: |
echo "Subverso versions:"
find . -name lake-manifest.json -print0 | xargs -0 jq '.packages[] | select(.name == "subverso") | {"file": input_filename, "subverso": .rev}'
find . -name lake-manifest.json -print0 | xargs -0 jq -e -s 'map(.packages[] | select(.name == "subverso").rev) | .[0] as $x | all(.[]; . == $x)'

0 comments on commit b84e7f6

Please sign in to comment.