Skip to content

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

@bcoebcoe commented Oct 9, 2020

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
bcoe requested review from BridgeAR, Trott and aduh95October 9, 2020 22:05
@nodejs-github-botnodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Oct 9, 2020
Comment threaddoc/api/deprecations.md Outdated
Comment threaddoc/api/deprecations.md Outdated
Benjamin E. Coeand others added 2 commits October 9, 2020 18:10
Co-authored-by: Anna Henningsen <github@addaleax.net>
@bcoe
bcoe requested review from Trott and addaleaxOctober 10, 2020 01:18
@nodejs-github-bot

This comment has been minimized.

@bcoebcoe added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 11, 2020
@nodejs-github-bot

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

bcoe commented Oct 12, 2020

Copy link
Copy Markdown
ContributorAuthor

Landed in c759281

@bcoebcoe closed this Oct 12, 2020
@bcoe
bcoe deleted the document-rmdir-recurse-deprecation branch October 12, 2020 16:46
@bcoebcoe mentioned this pull request Oct 12, 2020
4 tasks
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>
@MylesBorinsMylesBorins 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>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.deprecationsIssues and PRs related to deprecations.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@bcoe@nodejs-github-bot@iansu@Trott@addaleax@targos@aduh95