-
Notifications
You must be signed in to change notification settings - Fork 18
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
f445817
commit 10f066c
Showing
1 changed file
with
7 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -74,7 +74,6 @@ jobs: | |
needs: diff | ||
if: ${{ needs.diff.outputs.isRust == 'true' }} | ||
runs-on: ubuntu-ghcloud | ||
working-dir: ./site-builder | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: Swatinem/[email protected] | ||
|
@@ -85,30 +84,35 @@ jobs: | |
--rev 55ec89082466f6bb246d870a8d56d166a8e1f08b | ||
- name: Check formatting with rustfmt | ||
working-dir: ./site-builder | ||
run: > | ||
cargo fmt --all -- --check | ||
--config group_imports=StdExternalCrate,imports_granularity=Crate,imports_layout=HorizontalVertical | ||
- name: Check sorting of dependencies | ||
working-dir: ./site-builder | ||
run: cargo sort -w -c | ||
- name: Lint using clippy (w/o tests) | ||
working-dir: ./site-builder | ||
run: cargo clippy --all-features --no-deps -- -D warnings | ||
- name: Lint using clippy (w/ tests) | ||
working-dir: ./site-builder | ||
run: cargo clippy --all-features --tests --no-deps -- -D warnings | ||
- name: Check documentation | ||
working-dir: ./site-builder | ||
run: cargo doc --no-deps --workspace | ||
|
||
build: | ||
name: Build Rust code | ||
needs: diff | ||
if: ${{ needs.diff.outputs.isRust == 'true' }} | ||
runs-on: ubuntu-ghcloud | ||
working-dir: ./site-builder | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: Swatinem/[email protected] | ||
with: | ||
save-if: ${{ github.ref == 'refs/heads/main' && 'true' || 'false' }} | ||
- name: Build Rust code | ||
working-dir: ./site-builder | ||
run: cargo build --verbose | ||
|
||
# TODO(mlegner): Currently running all tests on all PRs touching Rust code. | ||
|
@@ -118,13 +122,13 @@ jobs: | |
needs: diff | ||
if: ${{ needs.diff.outputs.isRust == 'true' }} | ||
runs-on: ubuntu-ghcloud | ||
working-dir: ./site-builder | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: Swatinem/[email protected] | ||
with: | ||
save-if: ${{ github.ref == 'refs/heads/main' && 'true' || 'false' }} | ||
- name: Run tests | ||
working-dir: ./site-builder | ||
run: cargo test -- --include-ignored | ||
check-all: | ||
name: Check if all code checks succeeded | ||
|