search

AndyOGo / stylelint-declaration-strict-value

Specify properties for which a variable, function, keyword or value must be used.
MIT License
133 stars 10 forks source link

Question: it is neccessary to output console.error(error, messages.failedToFix(...)) in catch of autoFixFunc ? #276

Closed voskresla closed 1 year ago

voskresla commented 1 year ago

With this lines we get additional console error besides the report. When we pass throw null we get this null before default error

https://github.com/AndyOGo/stylelint-declaration-strict-value/blob/fc662943ede256727f4a5350f29cc27941073e84/src/index.ts#L442-L445

'scale-unlimited/declaration-strict-value': ['z-index', { 
    ignoreFunctions: false, 
    autoFixFunc: () => { throw null } 
}],

image

AndyOGo commented 1 year ago

That is unnecessary, yes

AndyOGo commented 1 year ago

@voskresla Thanks a lot for the issue.

I removed it an released v1.9.1