search

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

Fix #1446 #1457

Closed SukkaW closed 2 years ago

SukkaW commented 2 years ago

The PR fixed issue #1446.

All existing test cases pass, and a corresponding test case is added.

SukkaW commented 2 years ago

@ai That's weird, why did only the test on Node.js 17 fail?

Semigradsky commented 2 years ago

@SukkaW

ERROR: Coverage for lines (99.97%) does not meet global threshold (100%)

Coverage is calculated for Node.js 17 only.

By the way, it would be nice to add Node.js 18 to test worflow.

SukkaW commented 2 years ago

Coverage is calculated for Node.js 17 only.

Let me see what I can do. It seems that I forgot to add test cases for utils.isPureNumber.

By the way, it would be nice to add Node.js 18 to test workflow.

This should be done in a separate PR.

Update

The test cases for utils.isPureNumber is added.

ai commented 2 years ago

Thanks. Released in 10.4.5.