Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

rm: expose main functionality for nushell #5294

Closed
tertsdiepraam opened this issue Sep 20, 2023 · 1 comment
Closed

rm: expose main functionality for nushell #5294

tertsdiepraam opened this issue Sep 20, 2023 · 1 comment
Labels
good first issue For newcomers! U - rm

Comments

@tertsdiepraam
Copy link
Member

Part of: nushell/nushell#10447

nushell wants access to our rm internals. Which means exposing the remove function and Options struct. There should be documentation for all the public items in the API as well.

This is an excellent first issue: feel free to claim this!

We did the same thing for cp already. You could use that PR as a guide: #5152
@KAAtheWiseGit KAAtheWiseGit mentioned this issue Sep 22, 2023
Merged
@tertsdiepraam
Copy link
Member Author

Closed by #5304

@tertsdiepraam tertsdiepraam mentioned this issue Oct 7, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue For newcomers! U - rm
Projects
nushell integration
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tertsdiepraam