Skip to content

Incorporate getting-started content from book #314

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
May 26, 2020

Conversation

milesfrain
Copy link
Contributor

There's an accompanying PR (purescript-contrib/purescript-book#144) in the book repo to point to this guide as a first step in environment setup.

More discussion about this change can be read in: purescript-contrib/purescript-book#93

This address #309 in this repo.

Copy link
Collaborator

@hdgarrood hdgarrood left a comment

Choose a reason for hiding this comment

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

Thanks! Just a few comments.

@milesfrain
Copy link
Contributor Author

Thanks for the thorough review. Applied feedback in latest push.

Copy link
Collaborator

@hdgarrood hdgarrood left a comment

Choose a reason for hiding this comment

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

Thanks! I'll merge this in a day or so unless anyone else objects.

@hdgarrood
Copy link
Collaborator

Also, it's not a problem now, but would you mind pushing new commits rather than rebasing and force-pushing in the future? It's easier to review what's changed since the last time I looked that way, and GitHub already makes it easy to squash when it comes to merging.

@milesfrain
Copy link
Contributor Author

Yes, I agree. I believe this one was an incremental commit though.

@hdgarrood
Copy link
Collaborator

🤦 so it was, I just don't know how to use the GitHub UI apparently. Sorry!

@hdgarrood hdgarrood merged commit 75e3e75 into purescript:master May 26, 2020
@milesfrain milesfrain deleted the gs-book-update branch June 13, 2020 04:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants