-
-
Notifications
You must be signed in to change notification settings - Fork 410
update Floskell to 0.11.* #3933
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
update Floskell to 0.11.* #3933
Conversation
5e3b761 to
c3f0688
Compare
fendor
left a comment
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.
LGTM, just dont do the the version bumps :)
haskell-language-server.cabal
Outdated
| category: Development | ||
| name: haskell-language-server | ||
| version: 2.5.0.0 | ||
| version: 2.5.0.1 |
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.
HLS and all its plugins use lockstep versioning. Thus, all versions get universally bumped on release and you shouldn't bump versions in PRs, as this throws off our automation.
Just keep it at 2.5.0.0 :)
| cabal-version: 2.4 | ||
| name: hls-floskell-plugin | ||
| version: 2.5.0.0 | ||
| version: 2.5.1.0 |
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.
Just keep it as 2.5.0.0 :)
|
The Please also add yourself to CODEOWNERS, thanks! |
Floskell 0.11.* supports Aeson 2.2.*
674bc86 to
24d64da
Compare
|
Thanks @fendor and @michaelpj Can we squash and cherry-pick this onto https://github.com/haskell/haskell-language-server/tree/2.5.0.0 and release a revision of
Looking at |
|
We're not generally doing backports, so it'll just be in the next release. That will probably be soon, since there's a GHC release coming. |
https://www.stackage.org/lts-21.25 remove duplicate `extra-deps`
fec0f54 to
b81e2ff
Compare
|
this succeeds: |
michaelpj
left a comment
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.
Great stuff, thanks!
Closes: #3927
hls-floskell-pluginupdated tofloskell0.11.* . The reason is thatfloskell0.11.* supportsaeson2.2.* : https://hackage.haskell.org/package/floskell-0.11.0There is a small change required in
plugins/hls-floskell-plugin/src/Ide/Plugin/Floskell.hs. The easy change breaks compatibility withfloskell0.10. So I have bumped the version ofhls-floskell-pluginfrom2.5.0.0to2.5.1.0.haskell-language-server2.5.0.0 is pinned tohls-floskell-plugin2.5.0.0 exactly. This PR changes the bound onhls-floskell-pluginto2.5.*.haskell-language-server2.5.0.0haskell-language-server2.5.0.1.haskell-language-server2.5.0.1 is compatible with both:hls-floskell-plugin2.5.0.0 andfloskell0.10.*hls-floskell-plugin2.5.1.0 andfloskell0.11.*What do you think?
I suppose this PR should target the tag
2.5.0.0https://github.com/haskell/haskell-language-server/tree/2.5.0.0 instead ofmasterFirst test
git resettomaster. Apply only the change tohaskell-language-server.cabal. Buildhaskell-language-server2.5.0.1 and all other packages.This builds successfully:
Second test
In this PR, this builds successfully:
These tests were run in the
nix developshell.