stylelint

Using version 3.0.0 of the suitcss preprocessor, stylelint is reporting warnings for the normalize.css in the node_modules directory:

node_modules/normalize.css/normalize.css
12:3    ⚠  Unexpected vendor-prefix "-ms-text-size-adjust" (property-no-vendor-prefix) [stylelint]
13:3    ⚠  Unexpected vendor-prefix "-webkit-text-size-adjust" (property-no-vendor-prefix) [stylelint]
96:3    ⚠  Unexpected vendor-prefix "-webkit-text-decoration-skip" (property-no-vendor-prefix) [stylelint]
212:1   ⚠  Unexpected composition (selector-root-no-composition) [stylelint]
306:3   ⚠  Unexpected vendor-prefix "-webkit-appearance" (property-no-vendor-prefix) [stylelint]
392:3   ⚠  Unexpected vendor-prefix "-webkit-appearance" (property-no-vendor-prefix) [stylelint]
402:3   ⚠  Unexpected vendor-prefix "-webkit-appearance" (property-no-vendor-prefix) [stylelint]
409:3   ⚠  Unexpected vendor-prefix "::-webkit-input-placeholder" (selector-no-vendor-prefix) [stylelint]
420:3   ⚠  Unexpected vendor-prefix "-webkit-appearance" (property-no-vendor-prefix) [stylelint]

I was able to suppress these warnings by including a .stylelintignore file in my project. I was looking through the stylelint-config-suitcss docs, and it seems like the intent is for the node_modules directory to be ignored by default?

[giuseppeg]

So we’ve got 5 partecipants to this conversation.

Let’s vote: 👍 ignore node_modules 👎 keep the status-quo

[simonsmith]

For instance ESLint doesn’t lint dependencies

Based on that, perhaps we should make a change to ignore anything in node_modules? This issue seems to be causing hassle for users.