Skip to content

Commit

Permalink
ci: Split tests job into tests and lints (#1059)
Browse files Browse the repository at this point in the history
* ci: Split tests into tests and lint jobs

* Rename lint.yml to lints.yml

I see on Github that the job "lint" is still waiting while the Lints
jobs have completed with success so I wonder if it's because the
filename doesn't match the job name.

* commit
  • Loading branch information
NunoAlexandre authored Oct 27, 2022
1 parent 0032ab3 commit 1394d5b
Show file tree
Hide file tree
Showing 3 changed files with 50 additions and 2 deletions.
44 changes: 44 additions & 0 deletions .github/workflows/lints.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
on:
push:
branches: [main, 'release-v**']
pull_request:
name: Lints
jobs:
lints:
name: ${{ matrix.target }}
strategy:
matrix:
os: [ubuntu-latest]
target: [fmt, taplo, clippy]
runs-on: ${{ matrix.os }}
env:
RUST_TOOLCHAIN: "nightly-2022-05-09"
steps:
- name: Free space on Ubuntu
if: ${{ matrix.os }} == 'ubuntu-latest'
run: |
echo "Pre cleanup"
df -h
sudo rm -rf "/usr/local/share/boost"
sudo rm -rf "$AGENT_TOOLSDIRECTORY"
echo "Post cleanup"
df -h
- name: Check out code
uses: actions/checkout@v3
- name: Install latest nightly
uses: actions-rs/toolchain@88dc2356392166efad76775c878094f4e83ff746
with:
toolchain: ${{ env.RUST_TOOLCHAIN }}
default: true
- name: Install taplo
uses: actions-rs/[email protected]
with:
crate: taplo-cli
version: 0.7.2
use-tool-cache: true
- uses: Swatinem/rust-cache@cb2cf0cc7c5198d3364b9630e2c3d457f160790c
- name: Run lints
run: ./ci/script.sh
env:
TARGET: ${{ matrix.target }}

3 changes: 2 additions & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-latest]
target: [tests, integration, lint, clippy]
target: [tests, integration]
runs-on: ${{ matrix.os }}
env:
RUST_TOOLCHAIN: "nightly-2022-05-09"
Expand All @@ -30,6 +30,7 @@ jobs:
with:
toolchain: ${{ env.RUST_TOOLCHAIN }}
default: true
# We use Taplo to format our Cargo.toml files.
- name: Install taplo
uses: actions-rs/[email protected]
with:
Expand Down
5 changes: 4 additions & 1 deletion ci/script.sh
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,11 @@ case $TARGET in
RUST_MIN_STACK=8388608 cargo test --release --package runtime-integration-tests
;;

lint)
fmt)
cargo fmt -- --check
;;

taplo)
taplo fmt --check
;;

Expand Down

0 comments on commit 1394d5b

Please sign in to comment.