delta/node-new.git/.github, branch ofrobots-patch-1

doc: add multiple issue templates for GitHub

2018-08-12T17:18:40+00:00

Fixes: https://github.com/nodejs/node/issues/21812

PR-URL: https://github.com/nodejs/node/pull/22215
Reviewed-By: Rich Trott 
Reviewed-By: Gus Caplan 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: George Adams 
Reviewed-By: Trivikram Kamat 
Reviewed-By: James M Snell

meta: remove CODEOWNERS

2018-06-13T06:54:23+00:00

The CODEOWNERS file is not working out. It is not pinging any groups
other than TSC (and more-or-less can't), and it pings TSC mostly for all
the wrong things.

It was a valiant effort, and possible worth revisiting at a future date
when functionality is closer to what we need. But for now, let us bid it
adieu.

PR-URL: https://github.com/nodejs/node/pull/21161
Reviewed-By: Myles Borins 
Reviewed-By: Michael Dawson 
Reviewed-By: Ali Ijaz Sheikh 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Matteo Collina 
Reviewed-By: Tiancheng "Timothy" Gu 
Reviewed-By: Ujjwal Sharma

doc: add .github to CODEOWNERS

2018-05-16T21:03:29+00:00

Add .github directory to CODEOWNERS file.

PR-URL: https://github.com/nodejs/node/pull/20733
Reviewed-By: Gus Caplan 
Reviewed-By: Yuta Hiroto 
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: James M Snell 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Colin Ihrig 
Reviewed-By: Daniel Bevenius 
Reviewed-By: Michaël Zasso

doc: improve specificity in CODEOWNERS

2018-05-16T21:02:27+00:00

Update CODEOWNERS entry for README.md to only apply to the top level
README.md. For example, if test/common/README.md is updated, that should
not be assigned to TSC automatically.

PR-URL: https://github.com/nodejs/node/pull/20729
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: Gus Caplan 
Reviewed-By: Jon Moss 
Reviewed-By: Yuta Hiroto 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Daniel Bevenius 
Reviewed-By: Michaël Zasso 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: James M Snell

doc: reorder CODEOWNERS file

2018-05-16T05:29:28+00:00

Later rules take precedence over earlier rules, so move all rules that
assign ownership to TSC to the end of the file. There are likely other
reorderings that we will want to consider.

PR-URL: https://github.com/nodejs/node/pull/20732
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: James M Snell 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Colin Ihrig 
Reviewed-By: Daniel Bevenius 
Reviewed-By: Michaël Zasso 
Reviewed-By: Luigi Pinca 
Reviewed-By: Ruben Bridgewater

codeowners: add rule for *.gypi files

2018-05-16T04:11:04+00:00

PR-URL: https://github.com/nodejs/node/pull/20740
Reviewed-By: Michaël Zasso 
Reviewed-By: Luigi Pinca 
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat

codeowners: fix typo in v8-inspector team name

2018-05-16T04:10:30+00:00

PR-URL: https://github.com/nodejs/node/pull/20740
Reviewed-By: Michaël Zasso 
Reviewed-By: Luigi Pinca 
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat

meta: add initial CODEOWNERS file

2018-05-12T23:46:42+00:00

PR-URL: https://github.com/nodejs/node/pull/20554
Reviewed-By: Gus Caplan 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Michaël Zasso 
Reviewed-By: Matheus Marchini

doc: remove subsystem from pull request template

2018-03-06T08:33:16+00:00

Remove request that user provide a list of subsystems in the pull
request template.

* We already ask them to put the subsystems in the first line of the
  commit message.
* The subsystem is usually easy to determine.
* We have a bot that applies subsystem labels on new PRs and it seems to
  do a rather good job. Tools over rules. Let the bot do it.
* The fewer unnecessary things we ask for in the template, the lower the
  barrier to entry.
* Anecdotal, but I have never found it useful to have the person
  submitting the PR list out subsystems in the pull request post.

PR-URL: https://github.com/nodejs/node/pull/19125
Reviewed-By: Luigi Pinca 
Reviewed-By: Michaël Zasso 
Reviewed-By: Anatoli Papirovski 
Reviewed-By: Tobias Nießen 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Сковорода Никита Андреевич 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Gus Caplan 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Colin Ihrig 
Reviewed-By: Matheus Marchini 
Reviewed-By: Tiancheng "Timothy" Gu 
Reviewed-By: James M Snell 
Reviewed-By: Jon Moss 
Reviewed-By: Kyle Farnung

doc: split CONTRIBUTING.md

2018-01-23T19:23:40+00:00

PR-URL: https://github.com/nodejs/node/pull/18271
Fixes: https://github.com/nodejs/node/issues/17842
Reviewed-By: Michaël Zasso 
Reviewed-By: Luigi Pinca 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: James M Snell