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.

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

844 New sequence functions: names #855

Closed
wants to merge 1 commit into from

Conversation

ChristianGruen
Copy link
Contributor

Closes #844

@michaelhkay
Copy link
Contributor

I am having second thoughts about dropping items-starting-where, in particular (but also items-ending-where). I think items-starting-where probably has more use cases than items-after. Note also that items-after() can be simply derived as items-starting-where() => tail(), but it's less easy to derive items-starting-where(). I'm now inclined to keep these with the names subsequence-starting-where() and subsequence-ending-where().

@ChristianGruen
Copy link
Contributor Author

ChristianGruen commented Nov 24, 2023

I'm now inclined to keep these with the names subsequence-starting-where() and subsequence-ending-where().

…which really gets awkward, and we'd have array:subarray-starting-where.

I'd be very much in favor of resorting to fn:take-while and fn:drop-while/fn:skip-while, which are available in various other languages.

@michaelhkay michaelhkay added the Abandoned PR was rejected, withdrawn, or superseded label Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Abandoned PR was rejected, withdrawn, or superseded
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New sequence functions: names
2 participants