Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@slotThe
Copy link
Member

@slotThe slotThe commented Mar 19, 2022

NOTE: I will prepare the same thing for the xmonad repo, but don't want to create too much noise with PRs. After this is rubber-stamped I will just push it directly.

Description

  • Prefer GHC 8.10.7 to 8.10.4, as versions seem to have stabilised now.
  • Add support for Stackage LTS 19, which ships with GHC 9.0.2.
  • Since a new version of 9.2 has been released, prefer 9.2.2 over 9.2.1.

Also, explicitly pass -XCPP to hlint; see 1.

Checklist

  • I've read CONTRIBUTING.md

  • I've considered how to best test these changes (property, unit,
    manually, ...) and concluded: CI passes

  • [n/a] I updated the CHANGES.md file

@slotThe slotThe requested a review from liskin March 19, 2022 16:03
slotThe added a commit to slotThe/xmonad that referenced this pull request Mar 19, 2022
+ Prefer GHC 8.10.7 to 8.10.4, as versions seem to have stabilised now.
+ Add support for Stackage LTS 19, which ships with GHC 9.0.2.
+ Since a new version of 9.2 has been released, prefer 9.2.2 over 9.2.1.

Related: xmonad/xmonad-contrib#694
yaml: stack-master.yaml
- resolver: lts-19
ghc: 9.0.2
yaml: stack-master.yaml
Copy link
Member

Choose a reason for hiding this comment

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

Is this intentionally duplicated?


I should've added a comment back then about the stack/stack-master stuff: we primarily test using stack-master (uses X11, xmonad master) but we also want to make sure that it works with the latest X11, xmonad releases so we do that with the most and least recent LTS, just to make the test matrix a bit smaller (and thus less likely to hit github limits).

Copy link
Member Author

Choose a reason for hiding this comment

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

Whoops, not at all! Forgot to stack-master.yaml -> stack.yaml there

@liskin
Copy link
Member

liskin commented Mar 26, 2022

I'm a bit confused about why the hlint issue was not a problem before. Can anyone help me understand that? Otherwise this looks all right apart from the one issue in the stack workflow.

@slotThe
Copy link
Member Author

slotThe commented Mar 26, 2022

I'm a bit confused about why the hlint issue was not a problem before. Can anyone help me understand that?

It seems like the bug only fires when -XHaskell2010 is passed to hlint (see ndmitchell/hlint#1361). This was not done before, but the haskell-ci version that I built apparently adds it. I suppose another solution would be to remove XHaskell2010 altogether and let hlint figure out what the default extensions are.

@liskin
Copy link
Member

liskin commented Mar 26, 2022

It seems like the bug only fires when -XHaskell2010 is passed to hlint (see ndmitchell/hlint#1361). This was not done before, but the haskell-ci version that I built apparently adds it. I suppose another solution would be to remove XHaskell2010 altogether and let hlint figure out what the default extensions are.

Oh, I see. I was looking at the wrong diff. Will give it some more thought once I get back from pub :-)

@slotThe
Copy link
Member Author

slotThe commented Mar 29, 2022

@liskin got any insights from the beer? :)

+ Prefer GHC 8.10.7 to 8.10.4, as versions seem to have stabilised now.
+ Add support for Stackage LTS 19, which ships with GHC 9.0.2.
+ Since a new version of 9.2 has been released, prefer 9.2.2 over 9.2.1.

Also, explicitly pass -XCPP to hlint; see [1].

[1]: ndmitchell/hlint#1360
@liskin
Copy link
Member

liskin commented Apr 5, 2022

@liskin got any insights from the beer? :)

No real insights. I set a reminder for myself to check/drop the hlint patch in 2 months time (kinda hoped the fix would have been released by now) and forcepushed the stack.yaml fixes. Feel free to proceed with other repos, and sorry for the delay.

@liskin liskin merged commit 737f804 into xmonad:master Apr 5, 2022
slotThe added a commit to slotThe/xmonad that referenced this pull request Apr 5, 2022
+ Prefer GHC 8.10.7 to 8.10.4, as versions seem to have stabilised now.
+ Add support for Stackage LTS 19; this ships with GHC 9.0.2.
+ Since a new version of 9.2 has been released, prefer 9.2.2 over 9.2.1.

Related: xmonad/xmonad-contrib#694
slotThe added a commit to slotThe/X11 that referenced this pull request Apr 5, 2022
+ Prefer GHC 8.10.7 to 8.10.4, as versions seem to have stabilised now.
+ Add support for Stackage LTS 19; this ships with GHC 9.0.2.
+ Since a new version of 9.2 has been released, prefer 9.2.2 over 9.2.1.

Related: xmonad/xmonad-contrib#694
Related: xmonad/xmonad@a854cda
slotThe added a commit to slotThe/X11-xft that referenced this pull request Apr 5, 2022
+ Prefer GHC 8.10.7 to 8.10.4, as versions seem to have stabilised now.
+ Add support for Stackage LTS 19; this ships with GHC 9.0.2.
+ Since a new version of 9.2 has been released, prefer 9.2.2 over 9.2.1.

Related: xmonad/xmonad-contrib#694
Related: xmonad/xmonad@a854cda
Related: xmonad/xmonad-X11@bbc04982ecf9f36c2c706b2bb13c9b96a052aa19
@slotThe slotThe deleted the ci/update-ghcs branch October 25, 2022 06:51
fananchonun added a commit to fananchonun/xmonad that referenced this pull request Nov 17, 2022
+ Prefer GHC 8.10.7 to 8.10.4, as versions seem to have stabilised now.
+ Add support for Stackage LTS 19; this ships with GHC 9.0.2.
+ Since a new version of 9.2 has been released, prefer 9.2.2 over 9.2.1.

Related: xmonad/xmonad-contrib#694
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants