-
Notifications
You must be signed in to change notification settings - Fork 676
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
Add NoImplicitPrelude to buildTypeScript #9988
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.
Great!
3750653
to
1404b5b
Compare
1404b5b
to
912b97e
Compare
42162d6
to
4e5eb21
Compare
62f71c8
to
536f955
Compare
@sheaf: would you like a re-review, or do you just signal that the version now in the PR is different than reviewed? |
I'm waiting to see whether this new version passes CI. If it does then I will request re-review, otherwise I will roll it back. |
b83e978
to
4a84f6c
Compare
I'm re-requesting a review. The new change since the first version is that we also stop provisioning a |
This allows us to compile Setup.hs without depending on base. In particular, this ensures that a package with `build-type: Hooks` and a custom setup stanza that does not depend on base successfully compiles. Tested in PackageTests/SetupHooks/SetupHooksNoBase.
4a84f6c
to
42b8dc1
Compare
This allows us to compile Setup.hs without depending on base. In particular, this ensures that a package with
build-type: Hooks
and a custom setup stanza that does not depend on base successfully compiles.Tested in
PackageTests/SetupHooks/SetupHooksNoBase
.Template B: This PR does not modify behaviour or interface