Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
Site Readme: Link To More Advanced Workflow Doc #925
Conversation
|
Strong LGTM, thanks! |
|
@saivann wow, not sure I've ever received a strong LGTM before. :-) Thanks for reviewing! In the absence of critical feedback, this will be merged Monday. |
harding
added
the
Merge Scheduled
label
Jun 28, 2015
|
Thanks again for doing this, @harding - looks great. |
|
@harding I wish I had read about these features a while back :) So hopefully this can be helpful to any new contributor as well. |
harding
merged commit 1c817f0
into
bitcoin-dot-org:master
Jun 29, 2015
1 check passed
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
harding
added a commit
that referenced
this pull request
Jun 29, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
harding commentedJun 27, 2015
The Bitcoin.org readme covers the basics of opening a PR, but it doesn't document any sort of everyday workflow for regular contributors. I think documenting that directly in the readme might confuse newbies, so I documented it elsewhere and asked a couple contributors to review it.
They thought it was useful (thanks!), so this PR adds a link to it from the readme.