You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was looking at this issue for ammonite. The problem comes from fansi, because it would fail anything that contains \u001b or \u009b if that's not a part of a valid color. So, something like this will fail: fansi.Str("\u001b")
I understand that this was the idea, but maybe there's no need to fail on "invalid" strings like that? If fansi can't parse it, maybe let it be part of the string?
Please let me know what you think. I can also make a PR, if that makes sense.
The text was updated successfully, but these errors were encountered:
Hello!
I was looking at this issue for ammonite. The problem comes from fansi, because it would fail anything that contains
\u001b
or\u009b
if that's not a part of a valid color. So, something like this will fail:fansi.Str("\u001b")
I understand that this was the idea, but maybe there's no need to fail on "invalid" strings like that? If fansi can't parse it, maybe let it be part of the string?
Please let me know what you think. I can also make a PR, if that makes sense.
The text was updated successfully, but these errors were encountered: