Remove some TODOs

stylelint · Oct 16, 2023 · 6b9141e · 6b9141e
1 parent 2a1fc6d
commit 6b9141e
Show file tree

Hide file tree

Showing 9 changed files with 15 additions and 17 deletions.
diff --git a/lib/__tests__/disableRanges.test.mjs b/lib/__tests__/disableRanges.test.mjs
@@ -953,7 +953,7 @@ it('Less // line-disabling comment (with description)', async () => {
 	});
 });
 
-// eslint-disable-next-line jest/no-disabled-tests -- TODO: investigate the reason
+// eslint-disable-next-line jest/no-disabled-tests -- see #5348
 it.skip('Less // disable next-line comment (with multi-line description)', async () => {
 	const result = await testDisableRangesLess(
 		`a {

diff --git a/lib/reference/properties.js b/lib/reference/properties.js
@@ -533,8 +533,6 @@ const longhandSubPropertiesOfShorthandProperties = new Map([
 			'text-decoration-line',
 			'text-decoration-style',
 			'text-decoration-color',
-			// TODO: add support for text-decoration-thickness, Level 4
-			// https://w3c.github.io/csswg-drafts/css-text-decor-4/#text-decoration-width-property
 		]),
 	],
 	[

diff --git a/lib/rules/declaration-property-value-no-unknown/index.js b/lib/rules/declaration-property-value-no-unknown/index.js
@@ -194,9 +194,9 @@ const rule = (primary, secondaryOptions) => {
 };
 
 /**
- * TODO: This function avoids false positives because CSSTree doesn't fully support
- * some math functions like `clamp()` via `fork()`. In the future, it may be unnecessary.
  *
+ * @see csstree/csstree#164 min, max, clamp
+ * @see csstree/csstree#245 env
  * @see https://github.com/stylelint/stylelint/pull/6511#issuecomment-1412921062
  * @see https://github.com/stylelint/stylelint/issues/6635#issuecomment-1425787649
  *

diff --git a/lib/rules/selector-class-pattern/index.js b/lib/rules/selector-class-pattern/index.js
@@ -88,12 +88,13 @@ const rule = (primary, secondaryOptions) => {
 
 				const selector = String(classNode);
 
-				// TODO: `selector` may be resolved. So, getting its raw value may be pretty hard.
-				//       It means `endIndex` may be inaccurate (though non-standard selectors).
+				// `selector` may be resolved. So, getting its raw value may be pretty hard.
+				// It means `endIndex` may be inaccurate (though non-standard selectors).
 				//
-				//       For example, given ".abc { &_x {} }".
-				//       Then, an expected raw `selector` is "&_x",
-				//       but, an actual `selector` is ".abc_x".
+				// For example, given ".abc { &_x {} }".
+				// Then, an expected raw `selector` is "&_x",
+				// but, an actual `selector` is ".abc_x".
+				// see #6234
 				const endIndex = index + selector.length;
 
 				report({

diff --git a/lib/rules/selector-max-class/__tests__/index.mjs b/lib/rules/selector-max-class/__tests__/index.mjs
@@ -148,7 +148,7 @@ testRule({
 			column: 7,
 			endLine: 1,
 			endColumn: 26,
-			// TODO: The following case is actually correct. It's hard to get the original selector before resolved.
+			// see #6234
 			// endColumn: 23,
 		},
 		{
@@ -159,7 +159,7 @@ testRule({
 			column: 29,
 			endLine: 1,
 			endColumn: 43,
-			// TODO: The following case is actually correct. It's hard to get the original selector before resolved.
+			// see #6234
 			// endColumn: 40,
 		},
 	],

diff --git a/lib/rules/selector-max-combinators/__tests__/index.mjs b/lib/rules/selector-max-combinators/__tests__/index.mjs
@@ -189,7 +189,7 @@ testRule({
 			column: 19,
 			endLine: 1,
 			endColumn: 26,
-			// TODO: The following case is actually correct. It's hard to get the original selector before resolved.
+			// see #6234
 			// endColumn: 23,
 		},
 		{
@@ -200,7 +200,7 @@ testRule({
 			column: 7,
 			endLine: 1,
 			endColumn: 28,
-			// TODO: The following case is actually correct. It's hard to get the original selector before resolved.
+			// see #6234
 			// endColumn: 25,
 		},
 		{

diff --git a/lib/rules/selector-max-specificity/__tests__/index.mjs b/lib/rules/selector-max-specificity/__tests__/index.mjs
@@ -461,7 +461,7 @@ testRule({
 		},
 		{
 			code: ':nth-child(even of my-tag) {}',
-			skip: true, // TODO: We need to support `<complex-selector-list>` in `ignoreSelectors`.
+			skip: true, // see #6615
 		},
 	],
 

diff --git a/lib/rules/selector-max-specificity/index.js b/lib/rules/selector-max-specificity/index.js
@@ -127,7 +127,6 @@ const rule = (primary, secondaryOptions) => {
 			if (optionsMatches(secondaryOptions, 'ignoreSelectors', ownValue)) {
 				ownSpecificity = zeroSpecificity();
 			} else if (aNPlusBOfSNotationPseudoClasses.has(ownValue.replace(/^:/, ''))) {
-				// TODO: We need to support `<complex-selector-list>` in `ignoreSelectors`. E.g. `:nth-child(even of .foo)`.
 				return selectorSpecificity(node);
 			} else {
 				ownSpecificity = selectorSpecificity(node.clone({ nodes: [] }));

diff --git a/lib/utils/__tests__/isStandardSyntaxAtRule.test.mjs b/lib/utils/__tests__/isStandardSyntaxAtRule.test.mjs
@@ -45,7 +45,7 @@ describe('isStandardSyntaxAtRule', () => {
 		expect(isStandardSyntaxAtRule(atRule('@media(min-width: 100px) {};'))).toBeTruthy();
 	});
 
-	// eslint-disable-next-line jest/no-disabled-tests -- TODO: `postcss-sass` parser does not support `@mixin`.
+	// eslint-disable-next-line jest/no-disabled-tests -- see AleshaOleg/postcss-sass#56
 	it.skip('ignore `@content` inside mixins newline', () => {
 		const rules = sassAtRules('@mixin mixin()\n  @content');