numerous CI improvements =u=
- run clippy before all other tasks - deny on clippy warnings when running in CI - build and test both xdg and infer backends - make sure "fif -V" actually works
This commit is contained in:
parent
b12c42adbb
commit
b3ce5d3d46
2 changed files with 45 additions and 15 deletions
|
@ -11,23 +11,42 @@ cache:
|
||||||
default:
|
default:
|
||||||
before_script:
|
before_script:
|
||||||
- rustc --version
|
- rustc --version
|
||||||
|
- cargo version
|
||||||
|
|
||||||
stages:
|
stages:
|
||||||
|
- lint
|
||||||
- build
|
- build
|
||||||
- test
|
- test
|
||||||
|
- version
|
||||||
build:
|
|
||||||
stage: build
|
|
||||||
script:
|
|
||||||
- cargo build --verbose --locked
|
|
||||||
|
|
||||||
cargo-test:
|
|
||||||
stage: test
|
|
||||||
script:
|
|
||||||
cargo test --verbose --locked
|
|
||||||
|
|
||||||
clippy:
|
clippy:
|
||||||
stage: test
|
stage: lint
|
||||||
script:
|
script:
|
||||||
- rustup component add clippy
|
- rustup component add clippy
|
||||||
- ./clippy.sh
|
- cargo clippy --version
|
||||||
|
- ./clippy.sh ci
|
||||||
|
|
||||||
|
xdg-backend:
|
||||||
|
stage: build
|
||||||
|
script:
|
||||||
|
- cargo build --features=xdg-mime-backend --locked
|
||||||
|
|
||||||
|
mime-backend:
|
||||||
|
stage: build
|
||||||
|
script:
|
||||||
|
cargo build --features=infer-backend --locked
|
||||||
|
|
||||||
|
cargo-test-xdg:
|
||||||
|
stage: test
|
||||||
|
script:
|
||||||
|
cargo test --features=xdg-mime-backend --verbose --locked
|
||||||
|
|
||||||
|
cargo-test-infer:
|
||||||
|
stage: test
|
||||||
|
script:
|
||||||
|
cargo test --features=infer-backend --verbose --locked
|
||||||
|
|
||||||
|
fif-version:
|
||||||
|
stage: version
|
||||||
|
script:
|
||||||
|
cargo run -- -V
|
15
clippy.sh
15
clippy.sh
|
@ -1,5 +1,15 @@
|
||||||
#!/bin/bash
|
#!/bin/bash
|
||||||
fd -e rs -x touch {}
|
set -e
|
||||||
|
|
||||||
|
_extra=""
|
||||||
|
if [ "$1" == "ci" ]; then
|
||||||
|
# deny on warnings when running in CI
|
||||||
|
_extra="-Dwarnings"
|
||||||
|
fi
|
||||||
|
|
||||||
|
# allow find to fail
|
||||||
|
find . -name '*.rs' -exec touch "{}" \; || true
|
||||||
|
|
||||||
cargo clippy --all-features --tests -- \
|
cargo clippy --all-features --tests -- \
|
||||||
-W clippy::nursery \
|
-W clippy::nursery \
|
||||||
-W clippy::perf \
|
-W clippy::perf \
|
||||||
|
@ -14,7 +24,8 @@ cargo clippy --all-features --tests -- \
|
||||||
-A clippy::unused_io_amount \
|
-A clippy::unused_io_amount \
|
||||||
-A clippy::redundant_closure_for_method_calls \
|
-A clippy::redundant_closure_for_method_calls \
|
||||||
-A clippy::shadow_unrelated \
|
-A clippy::shadow_unrelated \
|
||||||
-A clippy::option_if_let_else
|
-A clippy::option_if_let_else \
|
||||||
|
"$_extra"
|
||||||
|
|
||||||
# ALLOWS:
|
# ALLOWS:
|
||||||
# unused_io_amount: there are two places where i want to read up to X bytes and i'm fine with getting less than that
|
# unused_io_amount: there are two places where i want to read up to X bytes and i'm fine with getting less than that
|
||||||
|
|
Loading…
Reference in a new issue