Allow for line-ignore option

[muenzpraeger]

It is a common scenario that CSS classes get constructed within a JavaScript getter.

It would be great to have an ignore option like // slds-ignore-next-line that allows the developer to disable validation, in the sense like ESLint or Prettier allow to ignore lines.

[kirupa]

This is a good suggestion! I will add it to our next sprint’s deliverable 😀

[doniarobinson-sf]

Can we think of another valid case for people to ignore lines? Because in the example above, if we let them ignore that line, we don't get a chance to validate that second token. They might also choose to ignore lines instead of fixing them when they receive their reports for what they've done wrong. Maybe I'm cynical, but I can see a lot ways that can go awry, and we actually want a heavy degree of control here. However, another way of resolving the issue listed above is to check for a token that contains a curly brace and not validate that.

[muenzpraeger]

The same applies to every other part where the SLDS plugin shows errors, also with a CSS class. The developer should always be in full control of what they fix, and what they want to ignore. That's how every other linting works.

[jonnyl]

@doniarobinson-sf Loading animation within a component, like Lightning Desktop, that changes seasonally is a great example of where we would like to allow this entire file to be ignored by SLDSValidation. Ping me for specifics on the test failure/use-case.

[eltoroit]

+1

[billyma]

We are working on this item, and will publish the updated behavior soon.

[SteffenBaumann]

Any updates on this one?

[LoganTann-pro]

Can we also add a setting to ignore files matching a pattern, eg by adding an eslint ignore file support ?

Currently, the forceignore files cannot be used as this might prevent the file we want to ignore to be deployed by sfdx. Changing the eslint ignore file does not affect the lwc validator.