Bug with rescript compilation

[ashton]

Hey! I've installed "@davesnx/styled-ppx": "^0.50.0", and when trying to run my project using rescript (10.1.4) with rescript-react (0.11.0), it shows the following error

It seems that the compilation of this line is generating triple quotes and thus, an invalid syntax in the .bs.js file.

[lubegasimon]

In the meantime, those can be fixed by manually escaping the quotes

[ashton]

Which quotes exactly? (version 0.40.1 is working perfectly)

[lubegasimon]

Which quotes exactly?

For example; case """ : to case "\"":

[lubegasimon]

(version 0.40.1 is working perfectly)

We shall investigate and fix what could be causing the regression. Thanks for the report

[ashton]

For example; case """: to case "\"":

but those are generated files, everytime I run a npm install this would be overwritten

[ashton]

Also, not sure if it's related, but the PPX itself is not being recognized by rescript:

@lubegasimon Do you want me to open another issue?

[lubegasimon]

Also, not sure if it's related, but the PPX itself is not being recognized by rescript:

@lubegasimon Do you want me to open another issue?

Yes, though it seems something is missing out during the installation. Does this -- https://styled-ppx.vercel.app/getting-started guide help?

[ashton]

I think I followed what the getting started said, not sure if something is off because is not a new installation but an upgrade:

[lubegasimon]

It seems that the compilation of this line is generating triple quotes and thus, an invalid syntax in the .bs.js file

No, one occurrence is at

styled-ppx/packages/css/native/shared/Css_AtomicTypes.ml

Line 2281 in b06c03a

| {js|"|js} | {js|'|js} -> value

[davesnx]

I believe the issue has been fixed in main, but didn't release a npm package yet. Let me try to reproduce and double-check ReScript and release a 0.51

[davesnx]

Updated the https://github.com/davesnx/try-styled-ppx repository with ReScript 10 and styled-ppx 0.53 working nicely