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

guides: add guides from the core repo #1105

Merged
merged 1 commit into from
Jan 24, 2017
Merged

Conversation

evanlucas
Copy link
Contributor

This adds 4 guides from the nodejs/node repo to the guides section of
the website. The documentation working group would like to make these
guides more visible for end users.

Ref: nodejs/node#10792

Note: these guides have been copied from https://github.com/nodejs/node/tree/master/doc/topics with slight modification to fit the format of the website. The changes made are as follows:

  • The website does not support console or txt languages in code blocks
  • The domain postmortem example had to be inlined in the document
  • Metadata was added to the top of the documents (some already had it)

We are still trying to determine the place to make these most visible, but has decided to go ahead and
get them moved to this repo and determine the accessibility portion at a later date.

/cc @nodejs/documentation

This adds 4 guides from the nodejs/node repo to the guides section of
the website. The documentation working group would like to make these
guides more visible for end users.

Ref: nodejs/node#10792
Copy link
Member

@gibfahn gibfahn left a comment

Choose a reason for hiding this comment

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

This matches what was in nodejs/node

@hiroppy hiroppy self-requested a review January 19, 2017 12:41
Copy link
Member

@lpinca lpinca left a comment

Choose a reason for hiding this comment

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

Rrubber-stamp LGTM

@evanlucas
Copy link
Contributor Author

I'll land tomorrow if no one has objections. Thanks!

@evanlucas evanlucas merged commit b9448cc into nodejs:master Jan 24, 2017
@evanlucas evanlucas deleted the guides branch January 24, 2017 11:45
evanlucas added a commit to evanlucas/node that referenced this pull request Jan 24, 2017
This commit removes the topics and guides that the documentation
working group has proposed added to the website. We want them to have
more visibility and believe that moving them to the website does that.

Ref: nodejs/nodejs.org#1105
Fixes: nodejs#10792
PR-URL: nodejs#10896
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 24, 2017
This commit removes the topics and guides that the documentation
working group has proposed added to the website. We want them to have
more visibility and believe that moving them to the website does that.

Ref: nodejs/nodejs.org#1105
Fixes: nodejs#10792
PR-URL: nodejs#10896
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 27, 2017
This commit removes the topics and guides that the documentation
working group has proposed added to the website. We want them to have
more visibility and believe that moving them to the website does that.

Ref: nodejs/nodejs.org#1105
Fixes: nodejs#10792
PR-URL: nodejs#10896
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants