Skip to content
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

Sir4ur0n/doc/cleanup hie #2311

Merged
merged 3 commits into from
Oct 29, 2021
Merged

Conversation

sir4ur0n
Copy link
Collaborator

@sir4ur0n sir4ur0n commented Oct 29, 2021

It seems that #1230 removed the top level hie-xxx.yaml files but documentation was not updated accordingly, thus confusing for new contributors.

Also, it seems that implicit configuration detects as Stack project, so I clarified how to configure to force Cabal usage.

I also took the liberty to slightly improve visibility of some documentation links, when using inline code with links, it's really hard to tell it is a link, e.g.:
image

Copy link
Member

@jneira jneira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many thanks for the cleaning up and improve docs, looks really great

@jneira jneira added the merge me Label to trigger pull request merge label Oct 29, 2021
@mergify mergify bot merged commit b018717 into haskell:master Oct 29, 2021
@sir4ur0n sir4ur0n deleted the sir4ur0n/doc/cleanupHie branch November 2, 2021 10:11
pepeiborra pushed a commit that referenced this pull request Nov 2, 2021
* Update documentation on using HIE with HLS

Followup of #1230

* Cleanup some documentation links that are not visible with inline code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Label to trigger pull request merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants