aboutsummaryrefslogtreecommitdiff
path: root/COLLABORATOR_GUIDE.md
AgeCommit message (Collapse)Author
2016-10-19doc: add ctc-review label informationRich Trott
PR-URL: https://github.com/nodejs/node/pull/9072 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
2016-10-13doc: explains why Reviewed-By is added in PRsjessicaquynh
Adds verbose reasons to the documentation on why the Reviewed-By metadata on a pull request is important. This was loosely mentioned as an issue in the referenced issue below, and answered by @addaleax. Ref: https://github.com/nodejs/node/issues/8893 PR-URL: https://github.com/nodejs/node/pull/9044 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
2016-08-04meta: clarify process for breaking changesRich Trott
Fixes: https://github.com/nodejs/node/issues/7848 PR-URL: https://github.com/nodejs/node/pull/7955 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com> Reviewed-By: Julien Gilli <jgilli@nodejs.org> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
2016-03-13collaborator_guide: clarify commit message rulesWyatt Preul
Italicize the full URL being required in metadata. PR-URL: https://github.com/nodejs/node/pull/5661 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James Snell <jasnell@gmail.com>
2016-02-22doc: update repo docs to use 'CTC'Alexis Campailla
PR-URL: https://github.com/nodejs/node/pull/5304 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
2015-11-11doc: update lts description in the collaborator guideJames M Snell
PR-URL: https://github.com/nodejs/node/pull/3668 Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
2015-10-28doc: add LTS info to COLLABORATOR_GUIDE.mdMyles Borins
There is currently no information in the Collaborators guide regarding LTS. This commit adds some basic copy explaining what LTS is, what is considered for LTS, and a simple way collaborators can help. Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Steven R. Loomis <srloomis@us.ibm.com> PR-URL: https://github.com/nodejs/node/pull/3442
2015-09-24doc: jenkins-iojs.nodesource.com -> ci.nodejs.orgMichał Gołębiowski
The Jenkins URL has changed from https://jenkins-iojs.nodesource.com to https://ci.nodejs.org. The former address now redirects to the latter. Some jobs has also changed names. PR-URL: https://github.com/nodejs/node/pull/2886 Reviewed-By: Rod Vagg <r@va.gg> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Roman Reiss <me@silverwind.io>
2015-08-23node: rename from io.js to nodecjihrig
This commit replaces instances of io.js with Node.js, based on the recent convergence. There are some remaining instances of io.js, related to build and the installer. Fixes: https://github.com/nodejs/node/issues/2361 PR-URL: https://github.com/nodejs/node/pull/2367 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com>
2015-06-17doc: mention CI in Collaborator GuideRich Trott
Add link to Jenkins server in Collaborator Guide. PR-URL: https://github.com/nodejs/io.js/pull/1995 Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
2015-06-17doc: add TOC links to Collaborator GuideRich Trott
PR-URL: https://github.com/nodejs/io.js/pull/1994 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
2015-06-14doc: copyedit COLLABORATOR_GUIDE.mdRich Trott
PR-URL: https://github.com/nodejs/io.js/pull/1964 Reviewed-By: Alex Kocharin <alex@kocharin.ru> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
2015-05-18doc: update links from iojs/io.js to nodejs/io.jsFrederic Hemberger
Replaced all links in doc and comments to point to the new org. PR-URL: https://github.com/nodejs/io.js/pull/1715 Reviewed-By: Roman Reiss <me@silverwind.io>
2015-04-23doc: update branch to masterRoman Reiss
Update the remaining markdown files to refer to the master branch. PR-URL: https://github.com/iojs/io.js/pull/1511 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
2015-04-16doc: Document forced pushing with gitJohan Bergström
Mention that we generally disallow forced pushes but allow it in trivial cases within 10 minutes of the original push unless the branch pushed to already has new commits. PR-URL: https://github.com/iojs/io.js/pull/1420 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
2015-04-09doc: update curl usage in COLLABORATOR_GUIDERoman Reiss
Github now redirects the patch urls hosted on github.com, making it necessary to enable curl redirect handling to obtain patches in the merge process. Example (before and after): https://github.com/iojs/io.js/pull/1382.patch https://patch-diff.githubusercontent.com/raw/iojs/io.js/pull/1382.patch PR-URL: https://github.com/iojs/io.js/pull/1382 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
2015-04-04doc: fix minor typos in COLLABORATOR_GUIDE.mdKelsey
PR-URL: https://github.com/iojs/io.js/pull/1320 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
2015-01-12doc: clarified & split up contribution docsRod Vagg
- clarified the role of "collaborators" - split out a governance doc - split out a collaborator guide - cleaned up the contributing doc - cleaned up the readme & added collaborators list PR-URL: https://github.com/iojs/io.js/pull/233 Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com> Reviewed-By: Mikeal Rogers <mikeal.rogers@gmail.com> (Note: no explicit "LGTM" from Mikeal) Discussed at TC meeting 2015-01-07, agreed to push further amendments to governance to future PRs.
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-13 13:22:45 +0000