-
Notifications
You must be signed in to change notification settings - Fork 4
Update Stack LTS to 19.33 (GHC 9.0.2) #53
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
base: main
Are you sure you want to change the base?
Changes from 7 commits
4d15bce
286bfbe
8704a6c
1216371
2cf9603
0b4cf95
3411d89
33161c7
6af939b
b2427a3
2dec50c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,20 +5,15 @@ on: | |
pull_request: { branches: [main] } | ||
create: { tags: [v*] } | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.ref }} | ||
cancel-in-progress: true | ||
|
||
defaults: | ||
run: | ||
shell: bash | ||
|
||
jobs: | ||
cancel: | ||
name: Cancel redundant actions already in progress | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Cancel actions in progress of same workflow and same branch | ||
uses: styfle/cancel-workflow-action@0.9.0 | ||
with: | ||
access_token: ${{ github.token }} | ||
|
||
# Check that Haskell code is formatted. | ||
code-formatter: | ||
runs-on: ubuntu-latest | ||
|
@@ -31,6 +26,7 @@ jobs: | |
runs-on: ${{ matrix.os }} | ||
needs: code-formatter | ||
strategy: | ||
fail-fast: false | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. With this, failure on one job in the matrix does not cause other jobs to fail. |
||
matrix: | ||
os: | ||
- ubuntu-latest | ||
|
@@ -44,14 +40,14 @@ jobs: | |
# If you change it, make sure to adjust lower bounds there to reflect the change. | ||
# Also, make sure to adjust tested-with field in package.yaml so that it contains | ||
# corresponding GHC version. | ||
stack-resolver: lts-18.21 | ||
stack-resolver: lts-19.33 | ||
|
||
steps: | ||
- name: Checkout the repo | ||
uses: actions/checkout@v2 | ||
uses: actions/checkout@v4 | ||
|
||
- name: Cache (Unix) | ||
uses: actions/cache@v2 | ||
uses: actions/cache@v4 | ||
if: runner.os == 'Linux' || runner.os == 'macOS' | ||
with: | ||
path: | | ||
|
@@ -77,7 +73,7 @@ jobs: | |
restore-keys: | | ||
haskell-${{ runner.os }}-${{ hashFiles('stack.yaml') }}- | ||
- name: Cache (Windows) | ||
uses: actions/cache@v2 | ||
uses: actions/cache@v4 | ||
if: runner.os == 'Windows' | ||
with: | ||
# C\:sr is where stack installs compiled dependencies. | ||
|
@@ -134,10 +130,10 @@ jobs: | |
stack path --stack-root | ||
stack ghc -- --version | ||
ghc --version | ||
- name: Build dependencies | ||
|
||
- name: Build dependencies | ||
run: stack --install-ghc test --only-dependencies | ||
|
||
- name: Build StrongPath & Run tests | ||
run: stack test | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,7 +15,7 @@ category: System, Filesystem, FilePath | |
|
||
description: Replacement for a FilePath that enables you to handle filepaths in your code in a type-safe manner. You can specify at type level if they are relative, absolute, file, directory, posix, windows, and even to which file or directory they point to or are relative to. | ||
|
||
tested-with: GHC == 8.10.7, GHC == 9.0.1 # lts-18.21, nightly-2022-01-04 | ||
tested-with: GHC == 9.0.2 # lts-19.33 | ||
|
||
dependencies: | ||
- base >= 4.7 && < 5 | ||
|
@@ -35,7 +35,7 @@ library: | |
- path >=0.9.2 && <0.10 | ||
- exceptions >=0.10 && <0.11 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If I got it right, the lower bounds should match the version set in The lower bounds did match that LTS for the most part, but they ignored minor (third digit) changes. For example:
Was this done on purpose as a way to "broaden" the range? |
||
- filepath >=1.4 && <1.5 | ||
- template-haskell >=2.16 && <2.18 | ||
- template-haskell >=2.17 && <2.18 | ||
- hashable >=1.3 && < 1.4 | ||
|
||
tests: | ||
|
@@ -52,7 +52,7 @@ tests: | |
- filepath | ||
- hashable >=1.3 && < 1.4 | ||
- tasty >=1.4 && <1.5 | ||
- tasty-hspec >=1.1 && <1.3 | ||
- tasty-hspec >=1.2 && <1.3 | ||
- tasty-quickcheck >=0.10 && <0.11 | ||
- tasty-discover >=4.2 && <4.3 | ||
- hspec >=2.7 && <2.10 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why did hspec allow for 3 breaking change versions in its range? Is it a case of an "adventurous upper bound?" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Check changelog. |
||
- hspec >=2.8 && <2.10 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,7 @@ | |
|
||
# NOTE: If you modify this field, make sure to update the tested-with | ||
# field in package.yaml to contain the corresponding GHC versions. | ||
resolver: nightly-2022-01-04 | ||
resolver: lts-19.33 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've set both this resolver and the one in This PR only updates GHC to 9.0.2 and I've mostly done it to get a hang of the procedure, so picking the right LTS versions is not yet important. Once I actually fully update StrongPath to 9.6.7, which ranges would you like to support and test for? In other words, which LTSs should I list in |
||
|
||
# User packages to be built. | ||
# Various formats can be used as shown in the example below. | ||
|
@@ -43,8 +43,6 @@ packages: | |
# - git: https://github.com/commercialhaskell/stack.git | ||
# commit: e7b331f14bcffb8367cd58fbfc8b40ec7642100a | ||
# | ||
extra-deps: | ||
- path-0.9.2 | ||
|
||
# Override default flag values for local packages and extra-deps | ||
# flags: {} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,12 @@ | ||
# This file was autogenerated by Stack. | ||
# You should not edit this file by hand. | ||
# For more information, please see the documentation at: | ||
# https://docs.haskellstack.org/en/stable/lock_files | ||
# https://docs.haskellstack.org/en/stable/topics/lock_files | ||
|
||
packages: [] | ||
snapshots: | ||
- original: nightly-2022-01-04 | ||
completed: | ||
url: https://raw.githubusercontent.com/commercialhaskell/stackage-snapshots/master/nightly/2022/1/4.yaml | ||
sha256: ea584edfba307b5a88b51fc1db7c0c1b7da5f714fd30d69699fc78e3e1ce5212 | ||
size: 623654 | ||
packages: | ||
- original: | ||
hackage: path-0.9.2 | ||
completed: | ||
pantry-tree: | ||
sha256: 2acf94a62daeeb0aee9b77d044ece55b5e03445b574e6980a2e84a5a514f5517 | ||
size: 1206 | ||
hackage: path-0.9.2@sha256:2f2a7f01737cd350b30381b619e1a862601c83f10ede4d6935f76f66e63ae0c7,3273 | ||
- completed: | ||
sha256: 6d1532d40621957a25bad5195bfca7938e8a06d923c91bc52aa0f3c41181f2d4 | ||
size: 619204 | ||
url: https://raw.githubusercontent.com/commercialhaskell/stackage-snapshots/master/lts/19/33.yaml | ||
original: lts-19.33 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This automatically cancels in progress jobs, making our
cancel
step below redundant.