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

Implement support for Seq #23

Open
langpavel opened this issue Mar 26, 2016 · 4 comments
Open

Implement support for Seq #23

langpavel opened this issue Mar 26, 2016 · 4 comments

Comments

@langpavel
Copy link
Collaborator

How Seq should be displayed?

@Diggsey
Copy link

Diggsey commented Apr 22, 2016

It should display up to the first N values, followed by an ellipsis if the sequence continues.

@langpavel
Copy link
Collaborator Author

Ok, does anybody need this? :-) I'm also afraid of side effects. Can iterating of Seq have udesired side effects?

@Diggsey
Copy link

Diggsey commented Apr 22, 2016

Iterating any of the containers could potentially have side-effects given the ability to define properties in JS. I think in this case it's unlikely given the nature of code using ImmutableJS (namely, very pure, functional code)

@langpavel
Copy link
Collaborator Author

I know, but Seq is generated by function unlike other structures backed by data..

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

No branches or pull requests

2 participants