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

Please make remove this notification: " Replace color-adjust to print-color-adjust." #1463

Closed michaelrodov closed 2 years ago

michaelrodov commented 2 years ago

Hi, can you please make this notification "Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated." as info or something but not a warning/error

This is 3rd level dependency for us and our build fails on CI because of this warning (warn > errors on CI)

Thanks

ai commented 2 years ago

Change your CI settings config or fix the dependency.

By the nature, it is a warning. Changing text to info to fix CI is a treating the symptoms of the disease rather the source of the problem.

michaelrodov commented 2 years ago

We are using CRA, and there's no way to disable this behavior without setting CI=false which is wrong

ai commented 2 years ago

The whole idea of thinking that a warning is an error on CI is to be sure that your code doesn’t have any problem.

Using deprecated CSS properties is a problem.

Hiding this warning do not solve the problem.

Avoid broken 3rd level dependency. Or send PR with fix. It will solve the source of the problem.