diff options
| author | Jonathan <jonathan@mullvad.net> | 2022-06-22 11:33:11 +0200 |
|---|---|---|
| committer | Jonathan <jonathan@mullvad.net> | 2022-06-30 15:05:07 +0200 |
| commit | 8bc09fec9be38cef5140a23d6b9492bcbbd60dc7 (patch) | |
| tree | 4368c3a89a1effab2ca745cc8a4df9b478071065 /.github | |
| parent | 3bf745cb30c9916eacde2b65449054f6007d48a4 (diff) | |
| download | mullvadvpn-8bc09fec9be38cef5140a23d6b9492bcbbd60dc7.tar.xz mullvadvpn-8bc09fec9be38cef5140a23d6b9492bcbbd60dc7.zip | |
Fix newly introduced clippy warnings
Most significantly included making the TunnelArgs struct bigger by
including most of the parameters that are used in the call-chain for
Wireguard tunnels. This makes the struct less arbitrary and keeps down
the amount of arguments to a manageable level.
Fix `is_offline` lint complaint
Nightly clippy (0.1.63) does not complain about this however stable
(0.1.61) does and as such we'll change the name it. The mutability did not seem to
be necessary but we agreed the name change is good anyway.
Diffstat (limited to '.github')
| -rw-r--r-- | .github/workflows/clippy.yml | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/.github/workflows/clippy.yml b/.github/workflows/clippy.yml index ac37e831dc..0b76562513 100644 --- a/.github/workflows/clippy.yml +++ b/.github/workflows/clippy.yml @@ -32,4 +32,6 @@ jobs: - name: Clippy check run: | - ./ci/check-clippy.sh + export RUSTFLAGS="--deny warnings" + source env.sh + time cargo clippy --locked --verbose |