-
Notifications
You must be signed in to change notification settings - Fork 205
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
Bump ghcide #3943
Bump ghcide #3943
Conversation
f415291
to
4646844
Compare
4646844
to
37f3d3f
Compare
@@ -128,7 +127,7 @@ build_docs_folder path versions = do | |||
-- maps, and here order matters. | |||
let versions_json = versions | |||
& map (\s -> "\"" <> s <> "\": \"" <> s <> "\"") | |||
& List.join ", " | |||
& List.intercalate ", " |
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.
I removed the dependency on MissingH
since we already depend on enough other kitchen sink libraries and it doesn’t buy us anything.
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.
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.
CI failure might be due to bazel build @ghcide//:ghcide
failing.
@aherrmann-da thanks! should be fixed now. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
3d99082
to
2ee4b31
Compare
I noticed all uses of this library had been removed by #3943, but the library itself wasn't.
I noticed all uses of this library had been removed by #3943, but the library itself wasn't.
Pull Request Checklist
CHANGELOG_BEGIN
andCHANGELOG_END
tags, if appropriateNOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with
/AzurePipelines run
totrigger the build.