Skip to content

Commit eb0bbdd

Browse files
authored
clippy: GitHub inline annotation (#405)
* clippy: Add script to format json output as github inline annotation Signed-off-by: Jerens Lensun <[email protected]> * clippy: Make github inline annotation only work on specific platform and feature Signed-off-by: Jerens Lensun <[email protected]> * clippy: refactor to composite actions for clippy-annotation.sh Signed-off-by: Jerens Lensun <[email protected]> * clippy: fix the wrong action name on clippy annotation Signed-off-by: Jerens Lensun <[email protected]> * clippy: fix the wrong path clippy-annotation.sh Signed-off-by: Jerens Lensun <[email protected]> * clippy: add changes based on PR review Signed-off-by: Jerens Lensun <[email protected]> * clippy: consolidate clippy into one composite action Signed-off-by: Jerens Lensun <[email protected]> * clippy: make clippy not cancel if fail Signed-off-by: Jerens Lensun <[email protected]> * clippy: test the same config on actions repo Signed-off-by: Jerens Lensun <[email protected]> * clippy: changes clippy annotation action to use from servo/actions repo Signed-off-by: Jerens Lensun <[email protected]> --------- Signed-off-by: Jerens Lensun <[email protected]>
1 parent d918d57 commit eb0bbdd

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

.github/workflows/main.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,10 @@ jobs:
3535
run: cargo fmt --check
3636

3737
- name: clippy
38-
run: cargo clippy --features "${{ matrix.features }}" --target ${{ matrix.platform.target }}
38+
uses: servo/actions/cargo-annotation@main
39+
with:
40+
cargo-command: clippy --features '${{ matrix.features }}' --target '${{ matrix.platform.target }}'
41+
with-annotation: ${{ matrix.platform.target == 'x86_64-unknown-linux-gnu' && matrix.features == '' }}
3942

4043
- name: Cargo test
4144
run: cargo test --features "${{ matrix.features }} enable-slow-tests" --target ${{ matrix.platform.target }}

0 commit comments

Comments
 (0)