Skip to content

Commit

Permalink
Remove ANN pragma
Browse files Browse the repository at this point in the history
However don't yet run HLint on CI due
ndmitchell/hlint#1200
  • Loading branch information
phadej authored and RyanGlScott committed Jan 11, 2021
1 parent 7845bc2 commit b78fa8e
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 4 deletions.
2 changes: 2 additions & 0 deletions .hlint.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- ignore: {name: Unused LANGUAGE pragma}
- ignore: {name: Avoid lambda}
2 changes: 2 additions & 0 deletions cabal.haskell-ci
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ jobs: 2
jobs-selection: any
install-dependencies: False
allow-failures: <7.3
-- https://github.com/ndmitchell/hlint/issues/1200
-- hlint: True
irc-channels: irc.freenode.org#haskell-lens
irc-if-in-origin-repo: True
error-missing-methods: none
Expand Down
4 changes: 0 additions & 4 deletions fast/Data/Reflection.hs
Original file line number Diff line number Diff line change
Expand Up @@ -154,10 +154,6 @@ import System.IO.Unsafe
import Unsafe.Coerce
#endif

#ifdef HLINT
{-# ANN module "HLint: ignore Avoid lambda" #-}
#endif

-- Due to https://gitlab.haskell.org/ghc/ghc/issues/16893, inlining
-- unsafeCoerce too aggressively can cause optimization to become unsound on
-- old versions of GHC. As a workaround, we mark unsafeCoerce-using definitions
Expand Down
1 change: 1 addition & 0 deletions reflection.cabal
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ tested-with: GHC == 7.0.4
, GHC == 8.10.1

extra-source-files:
.hlint.yaml
examples/reflection-examples.cabal
examples/LICENSE
examples/*.hs
Expand Down

0 comments on commit b78fa8e

Please sign in to comment.