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

doc: document rmdir/recursive deprecation by bcoe · Pull Request #35579 · nodejs/node · GitHub

/ node Public

doc: document rmdir/recursive deprecation - #35579

Closed
bcoe wants to merge 5 commits into
nodejs:masterfrom
bcoe:document-rmdir-recurse-deprecation
Closed

doc: document rmdir/recursive deprecation#35579
bcoe wants to merge 5 commits into
nodejs:masterfrom
bcoe:document-rmdir-recurse-deprecation

Conversation

bcoe commented Oct 9, 2020
edited
Loading

Copy link
Copy Markdown
Contributor

Document deprecation of rmdir/recursive permissive functionality.


As suggested by @aduh95, this adds the deprecation of the permissive behavior of rmdir/recursive as a docs only deprecation, in the deprecations.md.

We already updated the docs in #35171, to mention that the permissive behavior will one day be removed.

Refs: #35562

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

CC: @iansu

Document deprecation of rmdir/recursive permissive functionality
bcoe requested review from BridgeAR, Trott and aduh95 October 9, 2020 22:05
nodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Oct 9, 2020
Comment thread doc/api/deprecations.md Outdated
Comment thread doc/api/deprecations.md Outdated
Benjamin E. Coe and others added 2 commits October 9, 2020 18:10
Co-authored-by: Anna Henningsen <github@addaleax.net>
bcoe requested review from Trott and addaleax October 10, 2020 01:18

This comment has been minimized.

bcoe added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Oct 11, 2020

Copy link
Copy Markdown
Collaborator

bcoe pushed a commit that referenced this pull request Oct 12, 2020
Document deprecation of rmdir/recursive permissive functionality

PR-URL: #35579
Refs: #35562
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

bcoe commented Oct 12, 2020

Copy link
Copy Markdown
Contributor Author

Landed in c759281

bcoe closed this Oct 12, 2020
bcoe deleted the document-rmdir-recurse-deprecation branch October 12, 2020 16:46
MylesBorins pushed a commit that referenced this pull request Oct 14, 2020
Document deprecation of rmdir/recursive permissive functionality

PR-URL: #35579
Refs: #35562
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
MylesBorins mentioned this pull request Oct 14, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
Document deprecation of rmdir/recursive permissive functionality

PR-URL: nodejs#35579
Refs: nodejs#35562
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
jasnell pushed a commit that referenced this pull request Aug 12, 2021
PR-URL: #39728
Refs: #35579
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams pushed a commit that referenced this pull request Aug 16, 2021
PR-URL: #39728
Refs: #35579
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Sep 4, 2021
PR-URL: #39728
Refs: #35579
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
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
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 outstanding review comments, and a CI started. deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants


Back | FazBrowse Home | New Git URL