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

Improve docs for potential contributors #1137

Merged
merged 2 commits into from Aug 17, 2015

Conversation

Projects
None yet
2 participants
@wsmoak
Contributor

wsmoak commented Aug 17, 2015

Going through the README's to learn how to use the latest code on the master branch I noticed a couple of minor things.

I didn't immediately understand the --dev version must be inside Phoenix directory error message. It doesn't seem to have anything to do with the version, it's about the location of the newly-created project.

Also when pulling changes from upstream, it's good to push them up to your fork on GitHub.

josevalim added a commit that referenced this pull request Aug 17, 2015

Merge pull request #1137 from wsmoak/howto-updates
Improve docs for potential contributors

@josevalim josevalim merged commit 7a3077f into phoenixframework:master Aug 17, 2015

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment