Skip to content

Commit 6ce5edc

Browse files
3364: simplify ci scripts r=phansch a=matthiaskrgr This makes it easier to flip the switch on pedantic lints on the clippy codebase. Co-authored-by: Matthias Krüger <[email protected]>
2 parents 1652393 + a90084d commit 6ce5edc

File tree

2 files changed

+12
-16
lines changed

2 files changed

+12
-16
lines changed

ci/base-tests.sh

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -24,21 +24,20 @@ cd clippy_lints && cargo test && cd ..
2424
cd rustc_tools_util && cargo test && cd ..
2525
# check that the lint lists are up-to-date
2626
./util/update_lints.py -c
27-
mkdir -p ~/rust/cargo/bin
28-
cp target/debug/cargo-clippy ~/rust/cargo/bin/cargo-clippy
29-
cp target/debug/clippy-driver ~/rust/cargo/bin/clippy-driver
30-
rm ~/.cargo/bin/cargo-clippy
27+
28+
CLIPPY="`pwd`/target/debug/cargo-clippy clippy"
3129
# run clippy on its own codebase...
32-
PATH=$PATH:~/rust/cargo/bin cargo clippy --all-targets --all-features -- -D clippy::all -D clippy::internal
30+
${CLIPPY} --all-targets --all-features -- -D clippy::all -D clippy::internal
3331
# ... and some test directories
34-
cd clippy_workspace_tests && PATH=$PATH:~/rust/cargo/bin cargo clippy -- -D clippy::all && cd ..
35-
cd clippy_workspace_tests/src && PATH=$PATH:~/rust/cargo/bin cargo clippy -- -D clippy::all && cd ../..
36-
cd clippy_workspace_tests/subcrate && PATH=$PATH:~/rust/cargo/bin cargo clippy -- -D clippy::all && cd ../..
37-
cd clippy_workspace_tests/subcrate/src && PATH=$PATH:~/rust/cargo/bin cargo clippy -- -D clippy::all && cd ../../..
38-
cd clippy_dev && PATH=$PATH:~/rust/cargo/bin cargo clippy -- -D clippy::all && cd ..
39-
cd rustc_tools_util/ && PATH=$PATH:~/rust/cargo/bin cargo clippy -- -D clippy::all && cd ..
32+
for dir in clippy_workspace_tests clippy_workspace_tests/src clippy_workspace_tests/subcrate clippy_workspace_tests/subcrate/src clippy_dev rustc_tools_util
33+
do
34+
cd ${dir}
35+
${CLIPPY} -- -D clippy::all
36+
cd -
37+
done
38+
4039

4140
# test --manifest-path
42-
PATH=$PATH:~/rust/cargo/bin cargo clippy --manifest-path=clippy_workspace_tests/Cargo.toml -- -D clippy::all
43-
cd clippy_workspace_tests/subcrate && PATH=$PATH:~/rust/cargo/bin cargo clippy --manifest-path=../Cargo.toml -- -D clippy::all && cd ../..
41+
${CLIPPY} --manifest-path=clippy_workspace_tests/Cargo.toml -- -D clippy::all
42+
cd clippy_workspace_tests/subcrate && ${CLIPPY} --manifest-path=../Cargo.toml -- -D clippy::all && cd ../..
4443
set +x

ci/integration-tests.sh

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,6 @@ function check() {
2828
}
2929

3030
case ${INTEGRATION} in
31-
rust-lang/cargo)
32-
check
33-
;;
3431
*)
3532
check
3633
;;

0 commit comments

Comments
 (0)