From: Jeffrey Czyz <jkczyz@gmail.com>
Date: Mon, 2 Aug 2021 15:30:01 +0000 (-0500)
Subject: Fail linter on #[warn(clippy::try_err)]
X-Git-Tag: v0.0.100~14^2
X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=b4c3a33f2954514c64cfc151c80e47afff22b9de;p=rust-lightning

Fail linter on #[warn(clippy::try_err)]

Some heavily used macros are using ? directly on an Err. Using a return
is easier to read and removes hundreds of linter warnings.

https://rust-lang.github.io/rust-clippy/master/index.html#try_err
---

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 5b886e19b..100fbdfcf 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -262,4 +262,4 @@ jobs:
           rustup component add clippy
       - name: Run default clippy linting
         run: |
-          cargo clippy -- -Aclippy::erasing_op -Aclippy::never_loop -Aclippy::if_same_then_else
+          cargo clippy -- -Aclippy::erasing_op -Aclippy::never_loop -Aclippy::if_same_then_else -Dclippy::try_err