generator-chisel icon indicating copy to clipboard operation
generator-chisel copied to clipboard

Resultions for sanitize.css

Open marziolek opened this issue 4 years ago • 4 comments

package.json needs "resolutions": { "sanitize.css": "12.0.1" },

marziolek avatar Sep 15 '21 10:09 marziolek

Hi @luboskmetko,

I have added the sanitize.css to the resolutions you can see in this PR. There is one problem, it will create a problem for ppl who are not using yarn because npm doesn't support "resolutions".
Some solutions I've found is using this package for converting resolutions for npm users https://www.npmjs.com/package/npm-force-resolutions
or using this script from https://newbedev.com/npm-equivalent-of-yarn-resolutions. Unfortunately, in my opinion, these are not elegant solutions. You need to run for example npm update-resolutions, after each npm i.

In the new version of Postcss-normalizer, they have solved the issue. As I remember, we need to have postcss version 8 for the update. In the update-dep(#500) task, I've updated that, and this problem will be resolved when we merge the update 🙂

edenizk avatar Sep 19 '21 12:09 edenizk

@edenizk could you create a issue in postcss-normlize asking to backport the fix to v9 https://github.com/csstools/postcss-normalize/pull/60 ?

jakub300 avatar Sep 20 '21 00:09 jakub300

@jakub300, I think this is a related issue https://github.com/csstools/postcss-normalize/issues/62

edenizk avatar Sep 27 '21 21:09 edenizk

Hi all, I created @jakub300/postcss-normalize that should work fine and released new chisel version ([email protected])

jakub300 avatar Sep 29 '21 22:09 jakub300