Skip to content

Commit

Permalink
Add regex support for ignorePseudoClasses option of `selector-pseud…
Browse files Browse the repository at this point in the history
…o-class-no-unknown` (#6316)
  • Loading branch information
ybiquitous committed Sep 3, 2022
1 parent 0632f55 commit 68a43cb
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 8 deletions.
5 changes: 5 additions & 0 deletions .changeset/big-crabs-tap.md
@@ -0,0 +1,5 @@
---
"stylelint": minor
---

Added: regex support for `ignorePseudoClasses` option of `selector-pseudo-class-no-unknown`
10 changes: 5 additions & 5 deletions lib/rules/selector-pseudo-class-no-unknown/README.md
Expand Up @@ -58,27 +58,27 @@ input:-moz-placeholder {}

## Optional secondary options

### `ignorePseudoClasses: ["/regex/", "string"]`
### `ignorePseudoClasses: ["/regex/", /regex/, "non-regex"]`

Given:

```json
["/^my-/", "pseudo-class"]
["/^--my-/", "--pseudo-class"]
```

The following patterns are _not_ considered problems:

<!-- prettier-ignore -->
```css
a:pseudo-class {}
a:--my-pseudo {}
```

<!-- prettier-ignore -->
```css
a:my-pseudo {}
a:--my-other-pseudo {}
```

<!-- prettier-ignore -->
```css
a:my-other-pseudo {}
a:--pseudo-class {}
```
Expand Up @@ -275,7 +275,7 @@ testRule({

testRule({
ruleName,
config: [true, { ignorePseudoClasses: ['unknown', '/^my-/', '/^YOUR-/i'] }],
config: [true, { ignorePseudoClasses: ['unknown', '/^my-/', /^YOUR-/i] }],

accept: [
{
Expand Down
4 changes: 2 additions & 2 deletions lib/rules/selector-pseudo-class-no-unknown/index.js
Expand Up @@ -18,7 +18,7 @@ const report = require('../../utils/report');
const ruleMessages = require('../../utils/ruleMessages');
const validateOptions = require('../../utils/validateOptions');
const vendor = require('../../utils/vendor');
const { isString } = require('../../utils/validateTypes');
const { isString, isRegExp } = require('../../utils/validateTypes');
const { isAtRule } = require('../../utils/typeGuards');

const ruleName = 'selector-pseudo-class-no-unknown';
Expand All @@ -41,7 +41,7 @@ const rule = (primary, secondaryOptions) => {
{
actual: secondaryOptions,
possible: {
ignorePseudoClasses: [isString],
ignorePseudoClasses: [isString, isRegExp],
},
optional: true,
},
Expand Down

0 comments on commit 68a43cb

Please sign in to comment.