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

Update README.md #97

Closed
wants to merge 1 commit into from
Closed

Update README.md #97

wants to merge 1 commit into from

Conversation

travinkeith
Copy link
Contributor

Minor improvements to the text under "License" to improve the flow.

Minor improvements to the text under "License" to improve the flow.
Copy link

@josephNLD josephNLD left a comment

Choose a reason for hiding this comment

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

It's better, concise, and has Oxford commas.

@ftrader
Copy link
Contributor

ftrader commented Oct 18, 2017

I support this change, but we are are working on https://reviews.bitcoinabc.org for all change requests (these only get pushed to github). So we cannot directly merge it here.

@travinkeith , if you would please register an account at the ABC development site which I linked, or let us know if you don't wish to, then we can make this change to the README on your behalf. Thank you for your first contribution!

@travinkeith
Copy link
Contributor Author

Thanks @josephNLD and @ftrader .

I created an account on reviews.bitcoinabc.org now using my GitHub account, so the username is the same. I tried to go to https://reviews.bitcoinabc.org/source/bitcoin-abc/browse/master/doc/README.md but it seems that I cannot open it in editor myself and create a pull request. My apologies - I'm relatively a noob to Github as well.

And no problem. Happy to help. :)

@ftrader
Copy link
Contributor

ftrader commented Oct 20, 2017

@travinkeith : are you able to join btcforks slack via https://btcforks.signup.team/ ?

There are some more setup steps I can assist you with to get your change into ABC.

@travinkeith
Copy link
Contributor Author

travinkeith commented Oct 23, 2017

@ftrader Sure. I'll try to do that tomorrow. Was a bit sick last week and catching up with things. Need to crash at the moment.

@schancel
Copy link
Contributor

@travinkeith are you still active? I can help you create the patch if you'd like.

@travinkeith
Copy link
Contributor Author

Hi @schancel

I received further instructions on how to apply it, but got stuck on an endless loop using a Windows machine. I booted up my Linux machine a few weeks ago to try to get it to work (had quite a busy November) but got quite problematic. I should clarify that I'm not much of a developer.

Since this modification is quite minor, I'm happy to just have the modifications applied directly without a Github-like credit to me. Or is there an easy way about this?

@schancel
Copy link
Contributor

@travinkeith Sure, I can do that

@schancel
Copy link
Contributor

@schancel schancel closed this Dec 27, 2017
ftrader pushed a commit to bitcoin-cash-node/bitcoin-cash-node that referenced this pull request Aug 17, 2020
[ci] Transition to from only/except to 'rules' syntax

Closes Bitcoin-ABC#97

See merge request bitcoin-cash-node/bitcoin-cash-node!311
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants