-
-
Notifications
You must be signed in to change notification settings - Fork 368
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
update Floskell to 0.11.* #3933
Conversation
5e3b761
to
c3f0688
Compare
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
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
:)
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:
|
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-plugin
updated tofloskell
0.11.* . The reason is thatfloskell
0.11.* supportsaeson
2.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 withfloskell
0.10. So I have bumped the version ofhls-floskell-plugin
from2.5.0.0
to2.5.1.0
.haskell-language-server
2.5.0.0 is pinned tohls-floskell-plugin
2.5.0.0 exactly. This PR changes the bound onhls-floskell-plugin
to2.5.*
.haskell-language-server
2.5.0.0haskell-language-server
2.5.0.1.haskell-language-server
2.5.0.1 is compatible with both:hls-floskell-plugin
2.5.0.0 andfloskell
0.10.*hls-floskell-plugin
2.5.1.0 andfloskell
0.11.*What do you think?
I suppose this PR should target the tag
2.5.0.0
https://github.com/haskell/haskell-language-server/tree/2.5.0.0 instead ofmaster
First test
git reset
tomaster
. Apply only the change tohaskell-language-server.cabal
. Buildhaskell-language-server
2.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 develop
shell.