mirror of
https://github.com/nushell/nushell
synced 2024-12-26 13:03:07 +00:00
Remove clippy global -A
from CI (#10072)
# Description We allowed two default lints due to false positives. This should be locally addressed with explicit information what causes either a false positive or why a particular design choice in the code is made. # User-Facing Changes None # Tests + Formatting We should detect a few more things with clippy out of the box. If you encounter a false positive you will need to address it on a case by case basis.
This commit is contained in:
parent
bffd8e4dd2
commit
43ceb3edec
1 changed files with 1 additions and 1 deletions
2
.github/workflows/ci.yml
vendored
2
.github/workflows/ci.yml
vendored
|
@ -9,7 +9,7 @@ name: continuous-integration
|
||||||
env:
|
env:
|
||||||
NUSHELL_CARGO_TARGET: ci
|
NUSHELL_CARGO_TARGET: ci
|
||||||
NU_LOG_LEVEL: DEBUG
|
NU_LOG_LEVEL: DEBUG
|
||||||
CLIPPY_OPTIONS: "-D warnings -D clippy::unwrap_used -A clippy::needless_collect -A clippy::result_large_err"
|
CLIPPY_OPTIONS: "-D warnings -D clippy::unwrap_used"
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
fmt-clippy:
|
fmt-clippy:
|
||||||
|
|
Loading…
Reference in a new issue