-
Notifications
You must be signed in to change notification settings - Fork 691
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 a note for contributors installing fourmolu #10473
base: master
Are you sure you want to change the base?
Conversation
9ca9d0f
to
9e9abe5
Compare
9e9abe5
to
e2f28ea
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.
Trying to be a bit more formal, and include a way to install with a specific ghc. (Some people involved with cabal don't trust --with-compiler
, but I use it somewhat regularly.)
- Redundant "that". - Use indefinite article. - Assume ghc version installed via ghcup - Use `--with-compiler` option - Grammar, "its requirement" Co-Authored-By: brandon s allbery kf8nh <allbery.b@gmail.com>
b57438f
to
dab2b48
Compare
dab2b48
to
85ba5b8
Compare
85ba5b8
to
2493075
Compare
@geekosaur I've squashed your suggestions and moved the |
> ``` | ||
> [!Tip] | ||
> If you have multiple versions of GHC installed with `ghcup`, a series specific |
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.
> If you have multiple versions of GHC installed with `ghcup`, a series specific | |
> If you have multiple versions of GHC installed with `ghcup`, a series-specific |
Hyphens strike again...
> installing it this way is: | ||
> | ||
> ``` | ||
> $ cabal install fourmolu-0.12.0.0 --overwrite-policy=always --ignore-project |
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.
A different worry here is that people may have other versions of fourmolu installed, and may need them for other projects. Sadly, there is no good solution to this at present.
Fixes #10472.
Template B: This PR does not modify behaviour or interface
E.g. the PR only touches documentation or tests, does refactorings, etc.
Include the following checklist in your PR: