Skip to content

Commit

Permalink
Add CI checks for copy-packetline
Browse files Browse the repository at this point in the history
For the real purpose of this new check, which is to verify that
`just copy-packetline` has been run when needed, to ensure that
gix-packetline-blocking/src is not out of date, it should only be
necessary to run this check on one platform.

So this should probably be changed to `runs-on: ubuntu-latest`
instead of a matrix strategy soon. I'm running it on three platforms
initially to check the script (though some functionality of the
script is not exercised).

I've also tested the script locally on Ubuntu 22.04.4 LTS, and on
Windows 10 (in Git Bash).
  • Loading branch information
EliahKagan committed Apr 9, 2024
1 parent 48fdc51 commit 346d9ec
Showing 1 changed file with 32 additions and 0 deletions.
32 changes: 32 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,7 @@ jobs:
- uses: EmbarkStudios/cargo-deny-action@v1
with:
command: check ${{ matrix.checks }}

wasm:
name: WebAssembly
runs-on: ubuntu-latest
Expand All @@ -188,3 +189,34 @@ jobs:
name: crates with 'wasm' feature
- run: cd gix-pack && cargo build --all-features --target ${{ matrix.target }}
name: gix-pack with all features (including wasm)

check-packetline:
strategy:
matrix:
os:
- ubuntu-latest
- macos-latest
- windows-latest
continue-on-error: true
runs-on: ${{ matrix.os }}
defaults:
run:
shell: bash
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@v1
with:
toolchain: stable
- uses: extractions/setup-just@v2
- name: Check that working tree is initially clean
run: |
set -x
git status
git status | tee | grep -qF 'nothing to commit, working tree clean'
- name: Regenerate gix-packetline-blocking/src
run: just copy-packetline
- name: Check that gix-packetline-blocking/src was already up to date
run: |
set -x
git status
git status | tee | grep -qF 'nothing to commit, working tree clean'

0 comments on commit 346d9ec

Please sign in to comment.