Postcss: Deno vs PostCSS 8.1.7

Created on 10 Nov 2020  路  5Comments  路  Source: postcss/postcss

Recent ES imports TS fix killed Deno build.

@oscarotero we need your help

4777548-tec60feb1942

bug

All 5 comments

馃槃 mmm, weird. Let me investigate...

After spending some hours researching, I found that this error only happens in deno 1.5.1 and upper (in deno 1.5.0 works fine). And seems like this only happens by running the tests, but the library seems to work fine.

A workaround may be disabling the tests in the github action or, at least, force to install Deno 1.5.0 (it can be configured here: https://github.com/postcss/postcss/blob/master/.github/workflows/deno.yml#L16)

I'm going to create an issue in Deno repository.

Ok, I've updated the deno conversion to remove the changes introduced in 8.1.7, like CssSyntaxError.default = CssSyntaxError;

There's an open issue (https://github.com/denoland/deno/issues/8355) to fix this problem in Deno, but meanwhile I've removed here:
https://github.com/postcss/postcss-deno/blob/master/to_deno.js#L29

So, once this https://github.com/postcss/postcss/pull/1474 is merged, I think Deno conversion should work again.

Thanks!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

sydneyitguy picture sydneyitguy  路  5Comments

ZeeCoder picture ZeeCoder  路  7Comments

hudochenkov picture hudochenkov  路  12Comments

ben-eb picture ben-eb  路  11Comments

OEvgeny picture OEvgeny  路  9Comments