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

Fix typo in README.md #1985

Merged
merged 1 commit into from
Dec 13, 2019
Merged

Fix typo in README.md #1985

merged 1 commit into from
Dec 13, 2019

Conversation

suraneti
Copy link
Contributor

@suraneti suraneti commented Dec 3, 2019

Checklist
  • npm install && npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

Fix typo.
@gengjiawen gengjiawen merged commit 72592a9 into nodejs:master Dec 13, 2019
@gengjiawen
Copy link
Member

Thanks for the contribution :)

@suraneti suraneti deleted the bugfix/typo branch December 13, 2019 07:44
@rvagg
Copy link
Member

rvagg commented Dec 15, 2019

@gengjiawen prefix needs to be added to commit messages and messages should not start with upper-case, same rules as nodejs/node I think.
I'll fix this one up with a doc: fix typo ... and force push, so just for future reference. Thanks for staying on top though.

rvagg pushed a commit that referenced this pull request Dec 15, 2019
PR-URL: #1985
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
rvagg pushed a commit that referenced this pull request Dec 15, 2019
Running Python standalone tests on multiple OSes would free up Travis CI for tests of various combinations of Node.js and Python as well as tests on other [CPU architectures](https://docs.travis-ci.com/user/multi-cpu-architectures).  __arch: amd64, arm64, ppc64le, s390x__

PR-URL: #1985
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
rvagg pushed a commit that referenced this pull request Jan 3, 2020
PR-URL: #1985
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
rvagg pushed a commit that referenced this pull request Jan 3, 2020
Running Python standalone tests on multiple OSes would free up Travis CI for tests of various combinations of Node.js and Python as well as tests on other [CPU architectures](https://docs.travis-ci.com/user/multi-cpu-architectures).  __arch: amd64, arm64, ppc64le, s390x__

PR-URL: #1985
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
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.

None yet

4 participants