FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[v14.x backport] doc: refactor fs docs structure by targos · Pull Request #39706 · nodejs/node · GitHub

/ node Public

[v14.x backport] doc: refactor fs docs structure - #39706

Closed
targos wants to merge 6 commits into
nodejs:v14.x-stagingfrom
targos:restructure-fs-docs-v14
ConversationCommits (6)ChecksFiles changed
Closed

[v14.x backport] doc: refactor fs docs structure#39706
targos wants to merge 6 commits into
nodejs:v14.x-stagingfrom
targos:restructure-fs-docs-v14

Conversation

targos commented Aug 8, 2021

Copy link
Copy Markdown
Member
  • squash: make fs.md identical to master
  • squash: restore lost changes entry in dgram.md
  • doc: refactor fs docs structure
  • squash: revert bad changes from d963269
  • doc: change lang info string in fs JS snippets

github-actions Bot added doc Issues and PRs related to the documentations. v14.x labels Aug 8, 2021

targos commented Aug 8, 2021

Copy link
Copy Markdown
Member Author

Backport of #37170 and #37605

The first 4 commits are supposed to be squashed together.
Here's what I did:

  • Copy the version of fs.md from the master branch at the commit before the restructuring (creating d963269)
  • Fixed dgram.md (I realized the history entry was lost in a previous backport along with two others in fs.md)
  • Cherry-picked doc: refactor fs docs structure #37170 cleanly
  • Reverted the changes from d963269 that do not apply to v14.x or adapted them for v14.x

targos commented Aug 8, 2021

Copy link
Copy Markdown
Member Author

/cc @jasnell as you did the original changes. I haven't verified if everything that is in the new version of the docs applies to v14.x except for what is in d963269.

targos commented Aug 12, 2021

Copy link
Copy Markdown
Member Author

@nodejs/documentation @nodejs/fs

targos force-pushed the restructure-fs-docs-v14 branch from e5744a3 to fd0d821 Compare August 13, 2021 08:28

targos commented Aug 13, 2021

Copy link
Copy Markdown
Member Author

Anyone? Trying @nodejs/backporters

targos commented Aug 21, 2021

Copy link
Copy Markdown
Member Author

Maybe @nodejs/lts ?

targos added the release-agenda Issues and PRs to discuss during the meetings of the Release team. label Aug 25, 2021
targos mentioned this pull request Aug 25, 2021
Closed
BethGriggs added the review wanted PRs that need reviews. label Aug 26, 2021

targos commented Aug 29, 2021

Copy link
Copy Markdown
Member Author

/cc @nodejs/collaborators

See the explanation in #39706 (comment)
I'm not necessarily looking for a review of the diff (which is probably impossible).

Trott commented Aug 29, 2021

Copy link
Copy Markdown
Member

/cc @nodejs/collaborators

See the explanation in #39706 (comment)
I'm not necessarily looking for a review of the diff (which is probably impossible).

Did you mean to link to the comment in #39706 (comment)?

Trott requested a review from jasnell August 29, 2021 14:07

targos commented Aug 29, 2021

Copy link
Copy Markdown
Member Author

@Trott no

aduh95 reviewed Sep 1, 2021
View reviewed changes

aduh95 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I've only reviewed fs.md, found a few inconsistencies:

Comment thread doc/api/fs.md Outdated
Comment thread doc/api/fs.md Outdated
Comment thread doc/api/fs.md Outdated
Comment thread doc/api/fs.md Outdated
Comment thread doc/api/fs.md Outdated

targos commented Sep 1, 2021

Copy link
Copy Markdown
Member Author

@aduh95 okay, thank you. do these changes only apply to v14.x or should they also land on master?

targos closed this Sep 1, 2021
targos reopened this Sep 1, 2021
targos and others added 5 commits September 1, 2021 19:28
squash: make fs.md identical to master
2ed0c3f
squash: restore lost changes entry in dgram.md
136c98d
doc: refactor fs docs structure
6a5c4c5 
* Convert examples to esm to help promote that pattern with users
* Make Promises APIs more prominent to help promite that pattern
* Separate callback/sync apis into distinct sections to make those
  more consistent with the Promises api
* Improve other bits and pieces

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: nodejs#37170
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
squash: revert bad changes from d963269
29dafe9
doc: change lang info string in fs JS snippets
e912108 
Refs: nodejs/remark-preset-lint-node#176

PR-URL: nodejs#37605
Refs: nodejs#37162
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Danielle Adams <adamzdanielle@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
targos force-pushed the restructure-fs-docs-v14 branch from fd0d821 to e912108 Compare September 1, 2021 17:28

aduh95 commented Sep 1, 2021

Copy link
Copy Markdown
Contributor

@aduh95 okay, thank you. do these changes only apply to v14.x or should they also land on master?

These are the changes that are already on master and should be in this PR IMHO (either because those were part of #37170, or because they happened in a PR that has already been backported to v14.x-staging).

Apply suggestions from code review
2f751eb 
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
aduh95 mentioned this pull request Sep 1, 2021
Closed
targos added a commit that referenced this pull request Sep 1, 2021
doc: refactor fs docs structure
307c1d8 
* Convert examples to esm to help promote that pattern with users
* Make Promises APIs more prominent to help promite that pattern
* Separate callback/sync apis into distinct sections to make those
  more consistent with the Promises api
* Improve other bits and pieces

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #37170
Backport-PR-URL: #39706
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

targos commented Sep 1, 2021

Copy link
Copy Markdown
Member Author

Landed in 307c1d8 and f5b2fe1

targos closed this Sep 1, 2021
targos deleted the restructure-fs-docs-v14 branch September 1, 2021 21:16
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

aduh95 aduh95 approved these changes
jasnell Awaiting requested review from jasnell
+1 more reviewer
aymen94 aymen94 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

doc Issues and PRs related to the documentations. release-agenda Issues and PRs to discuss during the meetings of the Release team. review wanted PRs that need reviews.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

Back | FazBrowse Home | New Git URL