Skip to content

meta: document commit msg exception for long URLs - #20207

Closed
vsemozhetbyt wants to merge 1 commit into
nodejs:masterfrom
vsemozhetbyt:doc-commit-msg-long-urls
Closed

vsemozhetbyt wants to merge 1 commit into
nodejs:masterfrom
vsemozhetbyt:doc-commit-msg-long-urls

Conversation

@vsemozhetbyt

Copy link
Copy Markdown
Contributor
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Fixes: #17116
Refs: nodejs/core-validate-commit#24

@vsemozhetbyt vsemozhetbyt added meta Issues and PRs related to the general management of the project. fast-track PRs proposed for a shorter-than-standard waiting period before landing. labels Apr 22, 2018
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to Node.js documentation. label Apr 22, 2018
@vsemozhetbyt

Copy link
Copy Markdown
Contributor Author

Please, add 👍 here to approve fast-tracking.

@vsemozhetbyt

Copy link
Copy Markdown
Contributor Author

@BridgeAR BridgeAR added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Apr 22, 2018
vsemozhetbyt added a commit that referenced this pull request Apr 22, 2018
PR-URL: #20207
Fixes: #17116
Refs: nodejs/core-validate-commit#24
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
@vsemozhetbyt

Copy link
Copy Markdown
Contributor Author

Landed in 2060787

@vsemozhetbyt
vsemozhetbyt deleted the doc-commit-msg-long-urls branch April 22, 2018 10:26
jasnell pushed a commit that referenced this pull request Apr 23, 2018
PR-URL: #20207
Fixes: #17116
Refs: nodejs/core-validate-commit#24
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. doc Issues and PRs related to Node.js documentation. fast-track PRs proposed for a shorter-than-standard waiting period before landing. meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants