Refactor utilities to unify parseSelector() and parseSelectorAST()
#7647
Conversation
This unifies similar utilities using `postcss-selector-parser` into one. The new utility returns a root object for selectors.
|
| }); | ||
| }); | ||
|
|
||
| if (context.fix && fixedSelector) { |
There was a problem hiding this comment.
note
stylelint/lib/rules/selector-not-notation/__tests__/index.mjs
Lines 35 to 36 in 713f36f
could potentially return '' if it ever was supported.
| expect(selectorRoot).toHaveProperty('nodes[0].nodes[0].value', 'a'); | ||
| }); | ||
|
|
||
| expect(typeof processed).toBe('string'); |
There was a problem hiding this comment.
note
jest-extended has toBeString
|
Thanks for the review 👍🏼 |
stylelint#7647) This unifies similar utilities using `postcss-selector-parser` into one. The new utility returns a root object for selectors.
None.
This unifies similar utilities using
postcss-selector-parserinto one. The new utility returns a root object for selectors.Tip
I recommend ignoring whitespace changes for code review.