Replies: 3 comments
-
@alexesDev nice catch, imo it can be removed in order to get better performance, @activeadmin/collaborators what do u think ? |
Beta Was this translation helpful? Give feedback.
0 replies
-
Yeah, nice catch! I think the pretty HTML output is a nice feature that might be appreciated by some people, so it'd be nice to find a middle ground. Maybe make the "pretty html" an opt-out feature through configuration? |
Beta Was this translation helpful? Give feedback.
0 replies
-
I would accept a PR for this if you’re still interested in the change. Thank you. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi. I see
Arbre::HTML::Tag#indent
in stackprof top and it confuses me.I test this patch and got a 10% performance boost.
https://gist.github.com/alexesDev/343e22b45fac02fd7f509bb408874f66
Are indents really needed?
Beta Was this translation helpful? Give feedback.
All reactions