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 Include the release notes summary in the "Bug fixes" section notes:ready The "Release notes summary" section of this PR is ready to be included in our 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 Include the release notes summary in the "Bug fixes" section notes:ready The "Release notes summary" section of this PR is ready to be included in our 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