-
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
Upgrade to ghc-lib-8.8.1.20200225 #4692
Conversation
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.
Awesome. Thanks.
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.
Can we please have a test for daml 1.2
being optional?
@cocreature The next PR I raise will remove |
I prefer tests in the same PR that adds a new feature or introduces a change. |
3d61641
to
b0876f0
Compare
Two new files added:
|
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.
Thanks for adding the test, looks great 👍
63333e3
to
22a6caa
Compare
- Prelude `daml 1.2` is now optional; - `HsDumpAst` now comes from `ghc-lib-parser`. changelog_begin changelog_end
22a6caa
to
d678f3e
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
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.
Nice. Thanks.
@@ -227,7 +227,7 @@ basicTests mbScenarioService = Tasty.testGroup "Basic tests" | |||
setFilesOfInterest [a] | |||
expectWarning (a,0,25) "The import of ‘B’ is redundant" | |||
setBufferModified a "???" | |||
expectOneError (a,0,0) "Missing daml version" |
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 must be missing something: How was there a "Missing daml version" error in the past?
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.
well there is no DAML version if you replace the content by ???
🙂
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.
That is exactly the thing I missed. Thanks for pointing me at it.
daml 1.2
is now optional;HsDumpAst
now comes fromghc-lib-parser
.