postcss / autoprefixer

Parse CSS and add vendor prefixes to rules by Can I Use
https://twitter.com/autoprefixer
MIT License
21.58k stars 1.25k forks source link

Code changes to ease the conversion to Deno #1404

Closed oscarotero closed 3 years ago

oscarotero commented 3 years ago

Hi. I've working on adding Deno support for autoprefixer, something similar to postcss-deno.

You can see the first version here: https://github.com/lumeland/autoprefixer-deno

I had to do some changes to the code to ease this conversion (CommonJS to ESM). These changes don't affect to the external API, it's just internal code changes.

Let me know if it's ok or you want me to make any change. Thanks!

ai commented 3 years ago

Can I ask to use let for variables to keep my code style?

oscarotero commented 3 years ago

No problem, It's done.

Out of curiosity, can I know the reason of using let? const seems more appropiate if the variable won't be reassigned.

ai commented 3 years ago

Out of curiosity, can I know the reason of using let?

I want to have a simple rule of choosing variable type ☺

oscarotero commented 3 years ago

Thanks! I let you know when the Deno version is ready, in case you want me to transfer the repository to postcss organization.