opened
matrinox
username_0
Bug fix: Duplicate options before mutating it

Changes:
- calls to `inspector.awesome(x)` are replaced with `x.ai(@options)`

I noticed that ai calls `inspector.awesome` so I figured we can change it `#ai` without breaking much. I know this way is less efficient but for debugging, it's not going to be a huge concern.
Allow any object to override awesome print via #ai
issue
created
thiagofm
username_1
I quite like this change but I would like to hear other opnions on that.
comment
created
matrinox
username_0
@username_1 How do you guys do documentation? Let me know and I'll submit it as part of this PR
comment
created
matrinox
username_0
I'm happy with most of the code except one: the `top_layer` option. It's been a while since I've worked on this but from what I remember, I had to add the `top_layer` option to get around everything being tagged with `<pre>` tags after the root layer. Basically, I needed to separate the first call from the subsequent calls but I couldn't do that without more understanding; this was the simplest solution I could come up with.

I'm not too happy with it so I would love to hear how you guys would solve it.
comment