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

KT-26586: Sequence<T>.batch(n) operator implementation and documentation #1847

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@bradynpoulsen

bradynpoulsen commented Sep 4, 2018

https://youtrack.jetbrains.com/issue/KT-26586

The extension function was manually added to the generated _Sequences.kt file.
Does this need to be done elsewhere so it is properly generated?

@ilya-g

This comment has been minimized.

Show comment
Hide comment
@ilya-g

ilya-g Sep 4, 2018

Contributor

How does this relate to KT-10900 (Lazy Sequence concatenation)?

Contributor

ilya-g commented Sep 4, 2018

How does this relate to KT-10900 (Lazy Sequence concatenation)?

KT-26586 Sequence<T>.batch(n) operation implementation and documentation
The extension function was manually added to the generated _Sequences.kt file.
Does this need to be done elsewhere so it is properly generated?
@bradynpoulsen

This comment has been minimized.

Show comment
Hide comment
@bradynpoulsen

bradynpoulsen Sep 4, 2018

@ilya-g good catch! I didn't realize I had copied another issue number before starting my work. I updated the commit, let me know if you'd like me to re-open this with a corrected branch name as well

bradynpoulsen commented Sep 4, 2018

@ilya-g good catch! I didn't realize I had copied another issue number before starting my work. I updated the commit, let me know if you'd like me to re-open this with a corrected branch name as well

@bradynpoulsen bradynpoulsen changed the title from Sequence<T>.batch(n) operator implementation and documentation to KT-26586: Sequence<T>.batch(n) operator implementation and documentation Sep 4, 2018

@bradynpoulsen

This comment has been minimized.

Show comment
Hide comment
@bradynpoulsen

bradynpoulsen commented Sep 4, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment