aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Mueller <deso@posteo.net>2019-01-01 19:07:10 -0800
committerDaniel Mueller <deso@posteo.net>2019-01-01 19:07:10 -0800
commit686f61b75055ecb02baf9d9449525ae447a3bed1 (patch)
treecb8e9ff2b147877a188f5cb691d2137fa5e951e5
parent80eefafd3924ff8c8b1bd82d61d12b79b3369012 (diff)
downloadnitrocli-686f61b75055ecb02baf9d9449525ae447a3bed1.tar.gz
nitrocli-686f61b75055ecb02baf9d9449525ae447a3bed1.tar.bz2
Adjust clippy target in Gitlab CI
The clippy target as executed by the Gitlab CI excludes a bunch of lints when performing an initial run. That is necessary because some of the source code we rely on violates those rules and would cause the target to fail. The problem with the approach taken is that we list all the individual failing lints, which quickly becomes a maintenance burden. As it turns out clippy has the clippy::all meta-lint that subsumes all of the explicitly specified ones and so with this change we use that instead.
-rw-r--r--nitrocli/ci/gitlab-ci.yml3
1 files changed, 1 insertions, 2 deletions
diff --git a/nitrocli/ci/gitlab-ci.yml b/nitrocli/ci/gitlab-ci.yml
index b4ec21a..0b98522 100644
--- a/nitrocli/ci/gitlab-ci.yml
+++ b/nitrocli/ci/gitlab-ci.yml
@@ -23,8 +23,7 @@ lint:clippy:
# nitrocli (everything else is still up-to-date). That procedure is
# necessary because consumed dependencies may emit errors otherwise.
- cd nitrocli
- - cargo clippy --all-targets --all-features --
- -A warnings -A clippy::float_cmp -A clippy::cast_ptr_alignment -A clippy::unused_io_amount
+ - cargo clippy --all-targets --all-features -- -A clippy::all
- cargo clean --package=nitrocli
- cargo clippy --all-targets --all-features -- -D warnings