Stylish Haskell in CI

[newhoggy]

Run a stylish-haskell check in CI. There is one step which checks all tracked Haskell file, but it is not required. A second step checks all tracked Haskell files that have changed since the branch point vs `master, which is required.

There is a demo of if tailing the required check here:

https://github.com/input-output-hk/cardano-node/actions/runs/3947442222/jobs/6756264902

Sample output:

diff --git a/cardano-api/src/Cardano/Api/Fees.hs b/cardano-api/src/Cardano/Api/Fees.hs
index 138754a7..25b1d3ba 100644
--- a/cardano-api/src/Cardano/Api/Fees.hs
+++ b/cardano-api/src/Cardano/Api/Fees.hs
@@ -55,8 +55,8 @@ import           Data.Set (Set)
 import qualified Data.Set as Set
 import qualified Data.Text as Text
 import           GHC.Records (HasField (..))
-import           Numeric.Natural
 import           Lens.Micro ((^.))
+import           Numeric.Natural
 
 import           Control.Monad.Trans.Except
 import qualified Prettyprinter as PP
@@ -947,7 +947,7 @@ makeTransactionBodyAutoBalance eraInMode systemstart history pparams
     -- 4. balance the transaction and update tx change output
     txbody0 <-
       first TxBodyError $ createAndValidateTransactionBody txbodycontent
-        { txOuts = txOuts txbodycontent ++ 
+        { txOuts = txOuts txbodycontent ++
                    [TxOut changeaddr (lovelaceToTxOutValue 0) TxOutDatumNone ReferenceScriptNone]
             --TODO: think about the size of the change output
             -- 1,2,4 or 8 bytes?

[Jimbo4350]

I would get in touch with the benchmarking team before imposing stylish-haskell on them

[Jimbo4350]

I don't think the benchmarking team uses stylish-haskell or any style guide for that matter. cc: @kosyrevSerge

[newhoggy]

Yep, the include-list functionality is for them to be able to opt-out.