Skip to content

Commit 74f3c08

Browse files
authored
Fix at-rule-property-required-list message to use "descriptor" for accuracy (#8186)
Follow-up PR #8185 See also <#8148 (comment)> Note that it doesn't rename local variables to minimize code diffs and keep git-blame easy.
1 parent 63ea779 commit 74f3c08

File tree

3 files changed

+11
-2
lines changed

3 files changed

+11
-2
lines changed

.changeset/blue-carrots-share.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"stylelint": patch
3+
---
4+
5+
Fixed: `at-rule-property-required-list` message to use "descriptor" for accuracy

lib/rules/at-rule-property-required-list/index.cjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,12 @@ const ruleMessages = require('../../utils/ruleMessages.cjs');
1010
const validateObjectWithArrayProps = require('../../utils/validateObjectWithArrayProps.cjs');
1111
const validateOptions = require('../../utils/validateOptions.cjs');
1212

13+
// NOTE: We should have named this rule as `at-rule-descriptor-required-list` instead.
14+
// See https://github.com/stylelint/stylelint/pull/8185
1315
const ruleName = 'at-rule-property-required-list';
1416

1517
const messages = ruleMessages(ruleName, {
16-
expected: (atRule, property) => `Expected property "${property}" for at-rule "${atRule}"`,
18+
expected: (atRule, descriptor) => `Expected descriptor "${descriptor}" for at-rule "${atRule}"`,
1719
});
1820

1921
const meta = {

lib/rules/at-rule-property-required-list/index.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@ import ruleMessages from '../../utils/ruleMessages.mjs';
66
import validateObjectWithArrayProps from '../../utils/validateObjectWithArrayProps.mjs';
77
import validateOptions from '../../utils/validateOptions.mjs';
88

9+
// NOTE: We should have named this rule as `at-rule-descriptor-required-list` instead.
10+
// See https://github.com/stylelint/stylelint/pull/8185
911
const ruleName = 'at-rule-property-required-list';
1012

1113
const messages = ruleMessages(ruleName, {
12-
expected: (atRule, property) => `Expected property "${property}" for at-rule "${atRule}"`,
14+
expected: (atRule, descriptor) => `Expected descriptor "${descriptor}" for at-rule "${atRule}"`,
1315
});
1416

1517
const meta = {

0 commit comments

Comments
 (0)