Skip to content

Incorporate getting-started content from book #314

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

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants