Patches for BbML#240
Hidden character warning
Conversation
LPTK
left a comment
There was a problem hiding this comment.
LGTM. Plz address my one comment and I'll merge.
In a follow-up PR, it would be good to generate human-readable type variable names, instead of ones that include counters. For this, I think you can directly reuse the codegen.js.Scope helper. It's not really specific to JS, so we could move it out of this js sub-package. OTOH type pretty-printing could be considered a form of codegen, so Scope can remain in codegen.
Good suggestions! I will track this in a new issue. |
In this PR:
BBCtxand fix related bugsαeverywhere