Skip to content

rm: returns Error when it failed to remove a file#17237

Merged
WindSoilder merged 2 commits intonushell:mainfrom
WindSoilder:push-nxqzwntpqmtv
Dec 26, 2025
Merged

rm: returns Error when it failed to remove a file#17237
WindSoilder merged 2 commits intonushell:mainfrom
WindSoilder:push-nxqzwntpqmtv

Conversation

@WindSoilder
Copy link
Copy Markdown
Contributor

Fixes: #14900
The solution is suggested by @sgvictorino

Release notes summary - What our users need to know

rm command will return error when there is one file is failed to remove.

Tasks after submitting

@WindSoilder WindSoilder added notes:fixes Noted in "Bug fixes" section notes:ready Indicates Ready for Release notes labels Dec 25, 2025
@github-actions github-actions Bot added the A:std-library Defining and improving the standard library written in Nu label Dec 26, 2025
@WindSoilder WindSoilder merged commit ba5f920 into nushell:main Dec 26, 2025
16 checks passed
@github-actions github-actions Bot added this to the v0.110.0 milestone Dec 26, 2025
WindSoilder added a commit that referenced this pull request Dec 28, 2025
Fix CI failure after #17237
I don't know how can it happened, but it seems that `rm -f` inside tests
should fix the problem.

## Release notes summary - What our users need to know
NaN

## Tasks after submitting
@WindSoilder WindSoilder deleted the push-nxqzwntpqmtv branch January 29, 2026 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A:std-library Defining and improving the standard library written in Nu notes:fixes Noted in "Bug fixes" section notes:ready Indicates Ready for Release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rm command does not throw when removing file in use

1 participant