Skip to content

Commit

Permalink
Remove hlint from the CI and pre-commit config (#1759)
Browse files Browse the repository at this point in the history
  • Loading branch information
jonaprieto authored Jan 24, 2023
1 parent 807b3b1 commit 2094e5d
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 27 deletions.
13 changes: 0 additions & 13 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,19 +50,6 @@ jobs:
with:
clang-format-version: '15'
check-path: 'runtime/src'
hlint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: 'Set up HLint'
uses: haskell/actions/hlint-setup@v2
with:
version: '3.5'
- name: 'Run HLint'
uses: haskell/actions/hlint-run@v2
with:
path: '["src/", "app/", "test/"]'
# fail-on: warning

pre-commit:
runs-on: ubuntu-latest
Expand Down
16 changes: 8 additions & 8 deletions .hlint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@

- arguments: [--color]

# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------
# RESTRICTIONS
# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------

- extensions:
- default: false
Expand Down Expand Up @@ -50,15 +50,15 @@
- {name: Data.List.NonEmpty.nub, within: []}
- {name: Data.List.NonEmpty.nubBy, within: []}

# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------
# OTHER HINTS
# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------

# - warn: {name: Use explicit module export list}

# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------
# HINTS
# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------

- error: {lhs: idea Warning, rhs: warn}
- error: {lhs: idea Suggestion, rhs: suggest}
Expand All @@ -69,9 +69,9 @@
- error: {lhs: occNameString (occName x), rhs: occNameStr x}
- error: {lhs: noLoc (HsVar noExtField (noLoc (mkRdrUnqual (mkVarOcc x)))), rhs: strToVar x}

# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------
# IGNORES
# --------------------------------------------------------------------------------
# ------------------------------------------------------------------------------

- ignore: {name: Use let, within: [Test.All]}
- ignore: {name: Use String}
Expand Down
6 changes: 0 additions & 6 deletions .pre-commit-hooks.yaml
Original file line number Diff line number Diff line change
@@ -1,9 +1,3 @@
- id: hlint
name: hlint
description: Run Hlint linter
entry: make hlint
language: system

- id: ormolu
name: ormolu
description: Run Ormolu linter
Expand Down

0 comments on commit 2094e5d

Please sign in to comment.