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

browse() always shows ellipsis (...) #613

Closed
mduerig opened this issue May 23, 2017 · 5 comments
Closed

browse() always shows ellipsis (...) #613

mduerig opened this issue May 23, 2017 · 5 comments

Comments

@mduerig
Copy link

@mduerig mduerig commented May 23, 2017

Since 0.9 e.g. browse(List(1,2,3)) only shows ....

@lihaoyi
Copy link
Member

@lihaoyi lihaoyi commented May 23, 2017

Ahh yet another pprint regression.

browse should be changed to handle the default arguments the same way as show

@piotrkwiecinski
Copy link
Contributor

@piotrkwiecinski piotrkwiecinski commented May 23, 2017

I can grab this one if you haven't started @mduerig

@lihaoyi
Copy link
Member

@lihaoyi lihaoyi commented May 23, 2017

@piotrkwiecinski I got a fix out already, I think, lemme push it and see if CI is green. show was borked too in a similar way

@lihaoyi lihaoyi closed this in 0bfd399 May 23, 2017
@lihaoyi
Copy link
Member

@lihaoyi lihaoyi commented May 23, 2017

If it's green i'll tag it as 0.9.3

@mduerig
Copy link
Author

@mduerig mduerig commented May 24, 2017

Thanks for fixing this so quickly!

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

Successfully merging a pull request may close this issue.

None yet
3 participants