Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Conversation

@benhalverson
Copy link
Member

Description

Cherry picking staging branch contributing.md change I made in Feb

Related Issues

#538

@benhalverson benhalverson added contributing Relates to value added by awesome folks governance labels May 6, 2020
@designMoreWeb
Copy link
Contributor

LGTM

@benhalverson
Copy link
Member Author

@designMoreWeb If you want to approve this click on the tab files changed then review changes.

Copy link
Contributor

@designMoreWeb designMoreWeb left a comment

Choose a reason for hiding this comment

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

Looks good to me great work

@designMoreWeb
Copy link
Contributor

@designMoreWeb If you want to approve this click on the tab files changed then review changes.

Done

@benhalverson benhalverson merged commit 1c0c3e7 into nodejs:master May 7, 2020
* Push to your branch `git push -u origin yourbranch`
* Make a pull request against the remote branch `staging`
* Example `https://github.com/nodejs/nodejs.dev/pull/398`
* Mention @nodejs/website-redesign `/gcbrun`
Copy link
Contributor

Choose a reason for hiding this comment

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

I think here we need to use /preview instead of /gcbrun?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh ya forgot about that change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contributing Relates to value added by awesome folks governance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants