Skip to content

Add support for ghc-8.10.4 and drop it for ghc-8.10.1 #1331

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Feb 10, 2021
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
@@ -96,11 +96,6 @@ jobs:
- STACK_FILE: "stack-8.8.4.yaml"
<<: *defaults

ghc-8.10.1:
environment:
- STACK_FILE: "stack-8.10.1.yaml"
<<: *defaults

ghc-8.10.2:
environment:
- STACK_FILE: "stack-8.10.2.yaml"
@@ -111,6 +106,11 @@ jobs:
- STACK_FILE: "stack-8.10.3.yaml"
<<: *defaults

ghc-8.10.4:
environment:
- STACK_FILE: "stack-8.10.4.yaml"
<<: *defaults

ghc-default:
environment:
- STACK_FILE: "stack.yaml"
@@ -126,7 +126,7 @@ workflows:
- ghc-8.8.2
- ghc-8.8.3
- ghc-8.8.4
- ghc-8.10.1
- ghc-8.10.2
- ghc-8.10.3
- ghc-8.10.4
- ghc-default
14 changes: 7 additions & 7 deletions .github/mergify.yml
Original file line number Diff line number Diff line change
@@ -5,18 +5,18 @@ pull_request_rules:
method: squash
name: Automatically merge pull requests
conditions:
- status-success=bench-example (8.10.3, ubuntu-latest, Cabal-3.0.0.0)
- status-success=bench-example (8.10.3, ubuntu-latest, lsp-types-1.0.0.1)
- status-success=bench-example (8.10.4, ubuntu-latest, Cabal-3.0.0.0)
- status-success=bench-example (8.10.4, ubuntu-latest, lsp-types-1.0.0.1)

- status-success=nix (default, ubuntu-latest)
- status-success=nix (default, macOS-latest)

- status-success=test (8.10.4, ubuntu-latest)
- status-success=test (8.10.4, macOS-latest)
- status-success=test (8.10.3, ubuntu-latest)
- status-success=test (8.10.3, macOS-latest)
- status-success=test (8.10.2, ubuntu-latest)
- status-success=test (8.10.2, macOS-latest)
- status-success=test (8.10.1, ubuntu-latest)
- status-success=test (8.10.1, macOS-latest)
- status-success=test (8.8.4, ubuntu-latest)
- status-success=test (8.8.4, macOS-latest)
- status-success=test (8.8.3, ubuntu-latest)
@@ -27,15 +27,15 @@ pull_request_rules:
- status-success=test (8.6.5, macOS-latest)
- status-success=test (8.6.4, ubuntu-latest)
- status-success=test (8.6.4, macOS-latest)
- status-success=test (windows-latest, 8.10.3, true)
- status-success=test (windows-latest, 8.10.4, true)
- status-success=test (windows-latest, 8.10.3)
- status-success=test (windows-latest, 8.6.5, true)
- status-success=test (windows-latest, 8.10.2.2)
- status-success=test (windows-latest, 8.10.1)
# - status-success=test (windows-latest, 8.6.4)

- 'status-success=ci/circleci: ghc-8.10.4'
- 'status-success=ci/circleci: ghc-8.10.3'
- 'status-success=ci/circleci: ghc-8.10.2'
- 'status-success=ci/circleci: ghc-8.10.1'
- 'status-success=ci/circleci: ghc-8.8.4'
- 'status-success=ci/circleci: ghc-8.8.3'
- 'status-success=ci/circleci: ghc-8.8.2'
4 changes: 2 additions & 2 deletions .github/workflows/bench.yml
Original file line number Diff line number Diff line change
@@ -8,7 +8,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ghc: ['8.10.3']
ghc: ['8.10.4']
os: [ubuntu-latest]

steps:
@@ -76,7 +76,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ghc: ['8.10.3']
ghc: ['8.10.4']
os: [ubuntu-latest]
example: ['Cabal-3.0.0.0', 'lsp-types-1.0.0.1']

10 changes: 5 additions & 5 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
@@ -16,7 +16,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ghc: ['8.10.3', '8.10.2', '8.10.1', '8.8.4', '8.8.3', '8.8.2', '8.6.5', '8.6.4']
ghc: ['8.10.4', '8.10.3', '8.10.2', '8.8.4', '8.8.3', '8.8.2', '8.6.5', '8.6.4']
os: [ubuntu-latest, macOS-latest, windows-latest]
exclude:
- os: windows-latest
@@ -113,11 +113,11 @@ jobs:
path: ${{ steps.compress_server_binary.outputs.path }}

- name: Build Wrapper
if: matrix.ghc == '8.10.1'
if: matrix.ghc == '8.10.4'
run: cabal build exe:hls-wrapper -O2 $LINUX_CABAL_ARGS

- name: Compress Wrapper Binary
if: matrix.ghc == '8.10.1'
if: matrix.ghc == '8.10.4'
id: compress_wrapper_binary
run: |
HLS_WRAPPER_BUILD=$(find dist-newstyle \( -name 'hls-wrapper' -o -name 'hls-wrapper.exe' \) -type f)
@@ -136,7 +136,7 @@ jobs:
fi

- name: Upload Wrapper
if: matrix.ghc == '8.10.1'
if: matrix.ghc == '8.10.4'
uses: actions/[email protected]
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -147,7 +147,7 @@ jobs:
asset_content_type: ${{ steps.compress_wrapper_binary.outputs.content_type}}

- uses: actions/upload-artifact@v2
if: matrix.ghc == '8.10.1'
if: matrix.ghc == '8.10.4'
with:
name: haskell-language-server-wrapper-${{ runner.OS }}${{env.EXE_EXT}}.${{ steps.compress_wrapper_binary.outputs.extension }}
path: ${{ steps.compress_wrapper_binary.outputs.path }}
13 changes: 4 additions & 9 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -11,20 +11,15 @@ jobs:
strategy:
fail-fast: true
matrix:
ghc: ["8.10.3", "8.10.2", "8.10.1", "8.8.4", "8.8.3", "8.8.2", "8.6.5", "8.6.4"]
ghc: ["8.10.4", "8.10.3", "8.10.2", "8.8.4", "8.8.3", "8.8.2", "8.6.5", "8.6.4"]
os: [ubuntu-latest, macOS-latest]
include:
# one ghc-lib build
# should be renabled: #784
# - os: ubuntu-latest
# ghc: '8.10.1'
# ghc-lib: true
# only test supported ghc major versions
- os: ubuntu-latest
ghc: '8.10.3'
ghc: '8.10.4'
test: true
- os: windows-latest
ghc: '8.10.3'
ghc: '8.10.4'
test: true
- os: ubuntu-latest
ghc: '8.8.4'
@@ -39,7 +34,7 @@ jobs:
- os: windows-latest
ghc: '8.10.2.2'
- os: windows-latest
ghc: '8.10.1'
ghc: '8.10.3'
# This build get stuck frequently
# - os: windows-latest
# ghc: '8.6.4'
2 changes: 1 addition & 1 deletion cabal.project
Original file line number Diff line number Diff line change
@@ -25,7 +25,7 @@ package ghcide

write-ghc-environment-files: never

index-state: 2021-02-03T00:50:21Z
index-state: 2021-02-08T19:11:03Z

allow-newer:
active:base,
39 changes: 0 additions & 39 deletions ghcide/.azure/linux-stack.yml

This file was deleted.

41 changes: 0 additions & 41 deletions ghcide/.azure/windows-stack.yml

This file was deleted.

2 changes: 1 addition & 1 deletion ghcide/ghcide.cabal
Original file line number Diff line number Diff line change
@@ -13,7 +13,7 @@ description:
A library for building Haskell IDE's on top of the GHC API.
homepage: https://github.com/haskell/ghcide#readme
bug-reports: https://github.com/haskell/ghcide/issues
tested-with: GHC == 8.6.4 || == 8.6.5 || == 8.8.2 || == 8.8.3 || == 8.8.4 || == 8.10.1 || == 8.10.2 || == 8.10.3
tested-with: GHC == 8.6.4 || == 8.6.5 || == 8.8.2 || == 8.8.3 || == 8.8.4 || == 8.10.2 || == 8.10.3 || == 8.10.4
extra-source-files: include/ghc-api-version.h README.md CHANGELOG.md
test/data/hover/*.hs
test/data/multi/cabal.project
7 changes: 0 additions & 7 deletions ghcide/test/exe/Main.hs
Original file line number Diff line number Diff line change
@@ -3148,7 +3148,6 @@ checkFileCompiles fp diag =

pluginSimpleTests :: TestTree
pluginSimpleTests =
ignoreTest8101 "GHC #18070" $
ignoreInWindowsForGHC88And810 $
testSessionWithExtraFiles "plugin" "simple plugin" $ \dir -> do
_ <- openDoc (dir </> "KnownNat.hs") "haskell"
@@ -3163,7 +3162,6 @@ pluginSimpleTests =

pluginParsedResultTests :: TestTree
pluginParsedResultTests =
ignoreTest8101 "GHC #18070" $
ignoreInWindowsForGHC88And810 $
testSessionWithExtraFiles "plugin" "parsedResultAction plugin" $ \dir -> do
_ <- openDoc (dir</> "RecordDot.hs") "haskell"
@@ -4026,11 +4024,6 @@ pattern R x y x' y' = Range (Position x y) (Position x' y')
xfail :: TestTree -> String -> TestTree
xfail = flip expectFailBecause

ignoreTest8101 :: String -> TestTree -> TestTree
ignoreTest8101
| GHC_API_VERSION == ("8.10.1" :: String) = ignoreTestBecause
| otherwise = const id

ignoreInWindowsBecause :: String -> TestTree -> TestTree
ignoreInWindowsBecause = if isWindows then ignoreTestBecause else (\_ x -> x)

2 changes: 1 addition & 1 deletion haskell-language-server.cabal
Original file line number Diff line number Diff line change
@@ -14,7 +14,7 @@ copyright: The Haskell IDE Team
license: Apache-2.0
license-file: LICENSE
build-type: Simple
tested-with: GHC == 8.6.4 || == 8.6.5 || == 8.8.2 || == 8.8.3 || == 8.8.4 || == 8.10.1 || == 8.10.2 || == 8.10.3
tested-with: GHC == 8.6.4 || == 8.6.5 || == 8.8.2 || == 8.8.3 || == 8.8.4 || == 8.10.2 || == 8.10.3 || == 8.10.4
extra-source-files:
README.md
ChangeLog.md
2 changes: 1 addition & 1 deletion nix/default.nix
Original file line number Diff line number Diff line change
@@ -35,9 +35,9 @@ let
ourHaskell = pkgs.haskell // {
packages = pkgs.haskell.packages // {
# relax upper bounds on ghc 8.10.x versions (and skip running tests)
ghc8101 = extended (pkgs.haskell.packages.ghc8101.override sharedOverrides);
ghc8102 = extended (pkgs.haskell.packages.ghc8102.override sharedOverrides);
ghc8103 = extended (pkgs.haskell.packages.ghc8103.override sharedOverrides);
ghc8104 = extended (pkgs.haskell.packages.ghc8104.override sharedOverrides);
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure it this will work out of the box

};
};
};
4 changes: 1 addition & 3 deletions stack-8.10.3.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
resolver: nightly-2021-01-17
resolver: nightly-2021-02-06

packages:
- .
@@ -19,14 +19,12 @@ ghc-options:
"$everything": -haddock

extra-deps:
- apply-refact-0.9.0.0
- brittany-0.13.1.0
- Cabal-3.0.2.0
- clock-0.7.2
- data-tree-print-0.1.0.2@rev:2
- floskell-0.10.4
- fourmolu-0.3.0.0
- ghc-exactprint-0.6.3.4
- heapsize-0.3.0
- implicit-hie-cradle-0.3.0.2
- implicit-hie-0.1.2.5
47 changes: 18 additions & 29 deletions stack-8.10.1.yaml → stack-8.10.4.yaml
Original file line number Diff line number Diff line change
@@ -1,54 +1,43 @@
resolver: nightly-2020-08-16 # Last 8.10.1
resolver: nightly-2021-02-06
compiler: ghc-8.10.4

packages:
- .
- ./hie-compat
- ./ghcide/
# - ./shake-bench
- ./hls-plugin-api
- ./plugins/hls-class-plugin
- ./plugins/hls-haddock-comments-plugin
- ./plugins/hls-eval-plugin
- ./plugins/hls-explicit-imports-plugin
- ./plugins/hls-hlint-plugin
- ./plugins/hls-retrie-plugin
- ./plugins/hls-splice-plugin
- ./plugins/hls-tactics-plugin
- .
- ./hie-compat
- ./ghcide/
- ./hls-plugin-api
# - ./shake-bench
- ./plugins/hls-class-plugin
- ./plugins/hls-haddock-comments-plugin
- ./plugins/hls-eval-plugin
- ./plugins/hls-explicit-imports-plugin
- ./plugins/hls-hlint-plugin
- ./plugins/hls-retrie-plugin
- ./plugins/hls-splice-plugin
- ./plugins/hls-tactics-plugin

ghc-options:
"$everything": -haddock

extra-deps:
- apply-refact-0.9.0.0
- brittany-0.13.1.0
- Cabal-3.0.2.0
- clock-0.7.2
- data-tree-print-0.1.0.2@rev:2
- floskell-0.10.4
- fourmolu-0.3.0.0
- ghc-exactprint-0.6.3.4
- ghc-lib-8.10.3.20201220
- ghc-lib-parser-8.10.3.20201220
- haskell-lsp-0.23.0.0
- haskell-lsp-types-0.23.0.0
- heapsize-0.3.0
- hie-bios-0.7.1
- hlint-3.2.3
- HsYAML-aeson-0.2.0.0@rev:2
- implicit-hie-cradle-0.3.0.2
- implicit-hie-0.1.2.5
- lsp-test-0.12.0.0
- megaparsec-9.0.1
- monad-dijkstra-0.1.1.2
- opentelemetry-0.6.1
- opentelemetry-extra-0.6.1
- ormolu-0.1.4.1
- refinery-0.3.0.0
- retrie-0.1.1.1
- stylish-haskell-0.12.2.0
- semigroups-0.18.5
- temporary-1.2.1.1
- uniplate-1.6.13
- haskell-lsp-0.23.0.0
- haskell-lsp-types-0.23.0.0
- bytestring-encoding-0.1.0.0@sha256:460b49779fbf0112e8e2f1753c1ed9131eb18827600c298f4d6bb51c4e8c1c0d,1727
- hiedb-0.3.0.1

@@ -67,6 +56,6 @@ flags:
BuildExecutable: false

nix:
packages: [icu libcxx zlib]
packages: [ icu libcxx zlib ]

concurrent-tests: false
5 changes: 1 addition & 4 deletions stack-8.8.4.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
resolver: lts-16.25
resolver: lts-16.31 # last 8.8.4 lts

packages:
- .
@@ -29,8 +29,6 @@ extra-deps:
- floskell-0.10.4
- fourmolu-0.3.0.0
- ghc-exactprint-0.6.3.4
- ghc-lib-8.10.3.20201220
- ghc-lib-parser-8.10.3.20201220
- ghc-trace-events-0.1.2.1
- haskell-lsp-0.23.0.0
- haskell-lsp-types-0.23.0.0
@@ -54,7 +52,6 @@ extra-deps:
- semigroups-0.18.5
- stylish-haskell-0.12.2.0
- temporary-1.2.1.1
- uniplate-1.6.13
- bytestring-encoding-0.1.0.0@sha256:460b49779fbf0112e8e2f1753c1ed9131eb18827600c298f4d6bb51c4e8c1c0d,1727
- hiedb-0.3.0.1
- sqlite-simple-0.4.18.0@sha256:3ceea56375c0a3590c814e411a4eb86943f8d31b93b110ca159c90689b6b39e5,3002
4 changes: 2 additions & 2 deletions test/wrapper/Main.hs
Original file line number Diff line number Diff line change
@@ -18,8 +18,8 @@ main = do

projectGhcVersionTests :: TestTree
projectGhcVersionTests = testGroup "--project-ghc-version"
[ testCase "stack with ghc 8.10.1" $
testDir "test/wrapper/testdata/stack-8.10.1" "8.10.1"
[ testCase "stack with ghc 8.10.4" $
testDir "test/wrapper/testdata/stack-8.10.4" "8.10.4"
, testCase "stack with ghc 8.8.3" $
testDir "test/wrapper/testdata/stack-8.8.3" "8.8.3"
, testCase "cabal with global ghc" $ do
1 change: 0 additions & 1 deletion test/wrapper/testdata/stack-8.10.1/stack.yaml

This file was deleted.

File renamed without changes.
1 change: 1 addition & 0 deletions test/wrapper/testdata/stack-8.10.4/stack.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
resolver: ghc-8.10.4