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

meta: create github support file #27926

Closed
wants to merge 3 commits into from
Closed

Conversation

devsnek
Copy link
Member

@devsnek devsnek commented May 27, 2019

@nodejs/tsc @nodejs/community-committee

Refs: https://help.github.com/en/articles/adding-support-resources-to-your-project

Checklist

@devsnek devsnek added the meta Issues and PRs related to the general management of the project. label May 27, 2019
@Trott Trott self-assigned this May 27, 2019
Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

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

The shorthand links won't work without the references at the bottom. I'd inline all of them, though, so that people looking at this in a text editor can understand it easier.

@Trott
Copy link
Member

Trott commented May 27, 2019

(I'd also recommend replacing the information in README with a link to this file so we don't have to update information in two places.)

@Trott
Copy link
Member

Trott commented May 27, 2019

This is what I was going to propose, if it helps: master...Trott:support-file

Trott added a commit to Trott/io.js that referenced this pull request May 27, 2019
The GitHub interface will display links to SUPPORT.md in
hopefully-helpful places so let's take advantage of that and move the
support information to there and out of our README (but include a link
to the new SUPPORT.md).

Fixes: nodejs#27926
Refs: help.github.com/en/articles/adding-support-resources-to-your-project
@Trott Trott removed their assignment May 27, 2019
@Trott
Copy link
Member

Trott commented May 27, 2019

Looks like maybe the lite CI's didn't start automatically. Here's a manually-started Lite CI: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/3629/

@Trott Trott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 27, 2019
Copy link
Contributor

@keywordnew keywordnew left a comment

Choose a reason for hiding this comment

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

Looks great folks!

My non-blocking comment would be to add what the suggested resources are good for, so unfamiliar people can make better choices without as much guesswork.

For example:

* [Node.js Website](https://nodejs.org/en/)
Good for general information, links to latest changelogs, API docs.

Copy link
Contributor

@bnb bnb left a comment

Choose a reason for hiding this comment

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

Other than these small changes, LGTM.

.github/SUPPORT.md Show resolved Hide resolved
.github/SUPPORT.md Outdated Show resolved Hide resolved
@bnb
Copy link
Contributor

bnb commented May 27, 2019

Thank you @devsnek for PRng this in! 💖

Co-Authored-By: Tierney Cyren <accounts@bnb.im>
.github/SUPPORT.md Outdated Show resolved Hide resolved
Co-Authored-By: Rich Trott <rtrott@gmail.com>
Copy link
Member

@watilde watilde left a comment

Choose a reason for hiding this comment

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

<3

@Trott
Copy link
Member

Trott commented May 28, 2019

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

LGTM

@Trott
Copy link
Member

Trott commented May 30, 2019

Landed in f469522

@Trott Trott closed this May 30, 2019
Trott pushed a commit to Trott/io.js that referenced this pull request May 30, 2019
PR-URL: nodejs#27926
Refs: https://help.github.com/en/articles/adding-support-resources-to-your-project
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
targos pushed a commit that referenced this pull request May 31, 2019
PR-URL: #27926
Refs: https://help.github.com/en/articles/adding-support-resources-to-your-project
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
@devsnek devsnek deleted the support-file branch June 1, 2019 14:08
@targos targos mentioned this pull request Jun 3, 2019
@richardlau richardlau mentioned this pull request Jun 3, 2019
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. 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.

10 participants