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

clearer documentation #29

Merged
merged 10 commits into from
Oct 20, 2018
Merged

clearer documentation #29

merged 10 commits into from
Oct 20, 2018

Conversation

fommil
Copy link
Contributor

@fommil fommil commented Oct 19, 2018

I thought that the intro could do with a little bit of editing to make it more obvious what this script does, what it doesn't do, and get straight to the installation. Plus more links out to cabal documentation and features.

I also removed some redundancy (including the TODO section, which is already out of date with the issue tracker)

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@fommil
Copy link
Contributor Author

fommil commented Oct 19, 2018

@hasufell I pushed some fixes thanks to your feedback

README.md Outdated Show resolved Hide resolved
@hasufell
Copy link
Member

I thought that the intro could do with a little bit of editing to make it more obvious what this script does, what it doesn't do, and get straight to the installation. Plus more links out to cabal documentation and features.

I also removed some redundancy (including the TODO section, which is already out of date with the issue tracker)

Yes, links to cabal documentation sounds very useful, since this will probably mainly be used in conjunction with cabal new-build.

@fommil
Copy link
Contributor Author

fommil commented Oct 19, 2018

@hasufell some more updates, I hope all your comments are addressed now

@hasufell hasufell merged commit 3d51900 into haskell:master Oct 20, 2018
@fommil fommil deleted the with-compiler branch October 20, 2018 08:59
@fommil
Copy link
Contributor Author

fommil commented Oct 20, 2018

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants