-
Notifications
You must be signed in to change notification settings - Fork 63
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
Renders ETS as tuples #90
Renders ETS as tuples #90
Conversation
Given rows are always Elixir terms I wonder if it would make sense to always print them in |
@wojtekmach good call! Actually, I'm wondering if we should just always use monospaced font for the table, since we generally show inspected data. This way the styling would be consistent. |
@jonatanklosko I think monospaced functions will be bad for database data. We should definitely allow multiple representations there. It should be an option. |
Monospaced won't be good for Explorer either. :) |
@josevalim to be honest I think it looks better. Deepnote uses monospaced font for the tables too :) |
0fd33d7
to
1fdcd18
Compare
This PR changes the way we show ETS tables. Now we always have only one column named
rows
and all rows are just tuples.