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

Port the every command #626

Merged
merged 1 commit into from
Dec 30, 2021
Merged

Port the every command #626

merged 1 commit into from
Dec 30, 2021

Conversation

pantonshire
Copy link
Contributor

I wasn't completely sure how to handle negative strides; I'm silently casting them to usize because this is what the nth command does, but perhaps returning an error would be more appropriate?

@sophiajt
Copy link
Contributor

It's probably good for now - we'll probably circle back later and keep improving the errors as we go.

@pantonshire - if you want, you could file an issue so we remember to go back and improve the error later

@sophiajt sophiajt merged commit 822309b into nushell:main Dec 30, 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.

2 participants