fs: fix `fs.rm` support for loop symlinks by nathanael-ruf · Pull Request #45439 · nodejs/node

@nodejs-github-bot added fs

Issues and PRs related to the fs subsystem / file system.

needs-ci

PRs that need a full CI run.

labels

Nov 12, 2022

@Trott Trott added request-ci

Add this label to start a Jenkins CI on a PR.

and removed request-ci

Add this label to start a Jenkins CI on a PR.

labels

Nov 12, 2022

@aduh95 aduh95 changed the title fs: fix deleting invalid/loop symlinks. add tests. fs: fix fs.rm support for loop symlinks

Nov 12, 2022

aduh95

@nathanael-ruf

@aduh95 aduh95 added author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

request-ci

Add this label to start a Jenkins CI on a PR.

labels

Nov 13, 2022

LiviaMedeiros

@nathanael-ruf

@nathanael-ruf

LiviaMedeiros

@nathanael-ruf @LiviaMedeiros

Co-authored-by: Livia Medeiros <livia@cirno.name>

LiviaMedeiros

JungMinu

ruyadorno pushed a commit that referenced this pull request

Nov 21, 2022
Fixes: #45404
PR-URL: #45439
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>

marco-ippolito pushed a commit to marco-ippolito/node that referenced this pull request

Nov 23, 2022
Fixes: nodejs#45404
PR-URL: nodejs#45439
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>

danielleadams pushed a commit that referenced this pull request

Dec 30, 2022
Fixes: #45404
PR-URL: #45439
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>

danielleadams pushed a commit that referenced this pull request

Dec 30, 2022
Fixes: #45404
PR-URL: #45439
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>

danielleadams pushed a commit that referenced this pull request

Jan 3, 2023
Fixes: #45404
PR-URL: #45439
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>

danielleadams pushed a commit that referenced this pull request

Jan 4, 2023
Fixes: #45404
PR-URL: #45439
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>