Skip to content
This repository has been archived by the owner on Feb 8, 2022. It is now read-only.

Port keep, keep while and keep until commands #384

Merged
merged 4 commits into from Dec 17, 2021
Merged

Port keep, keep while and keep until commands #384

merged 4 commits into from Dec 17, 2021

Conversation

ghost
Copy link

@ghost ghost commented Nov 29, 2021

keep n was not listed in #242. I assume it's because first(n) would do the same, so I didn't implement it.

@sophiajt
Copy link
Contributor

Can we also have a basic keep command so you can do help keep and see the subcommands? We have that in a few places like help str and help skip.

@ghost
Copy link
Author

ghost commented Nov 30, 2021

Can we also have a basic keep command so you can do help keep and see the subcommands?

Yep, I'll add it ASAP 😄

@fdncred fdncred marked this pull request as draft December 10, 2021 00:17
@fdncred
Copy link
Contributor

fdncred commented Dec 10, 2021

converted to draft until the requested changes are made. thanks for the work so far!

@fdncred
Copy link
Contributor

fdncred commented Dec 15, 2021

@arthur-targaryen are you going to be able to finish this up or are you too busy at the moment?

@ghost
Copy link
Author

ghost commented Dec 16, 2021

@arthur-targaryen are you going to be able to finish this up or are you too busy at the moment?

Sorry I've been a bit busy. I'll find the time to finish this today!

@ghost ghost changed the title Port KeepWhile and KeepUntil commands Port keep, keep while and keep until commands Dec 16, 2021
@ghost ghost marked this pull request as ready for review December 16, 2021 22:23
@sophiajt
Copy link
Contributor

Looks good!

@sophiajt sophiajt merged commit 342584e into nushell:main Dec 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants