cleanup redundant upstream comments

Author: johanrd

lib/rules/template-link-rel-noopener.js

@@ -42,7 +42,6 @@ module.exports = {
             fix(fixer) {
               if (relAttr && relAttr.value?.type === 'GlimmerTextNode') {
                 // Strip existing noopener/noreferrer tokens, then re-add in canonical order
-                // (matches ember-template-lint behavior)
                 const oldValue = relAttr.value.chars.trim().replaceAll(/\s+/g, ' ');
                 const filtered = oldValue
                   .split(' ')

lib/rules/template-no-curly-component-invocation.js

@@ -126,7 +126,6 @@ module.exports = {
 
     /**
      * Build a fix function for block statement curly→angle bracket conversion.
-     * Matches ember-template-lint's fixForBlockNode behavior.
      */
     function buildBlockFix(node, angleBracketName) {
       return function fix(fixer) {

lib/rules/template-require-context-role.js

@@ -100,7 +100,7 @@ function hasAriaHiddenTrue(node) {
  * layer. Returns { ariaHidden, role } where:
  *   - `ariaHidden` is true if aria-hidden="true" was seen on any traversed
  *     element (including transparent wrappers) up to and including the first
- *     non-transparent parent — matches upstream's semantics.
+ *     non-transparent parent.
  *   - `role` is the role of the first non-transparent parent: a role string,
  *     null (element with no role), or undefined (no non-transparent parent).
  */

tests/lib/rules/template-no-heading-inside-button.js

@@ -12,7 +12,6 @@ ruleTester.run('template-no-heading-inside-button', rule, {
     '<template><h1>Title</h1></template>',
     '<template><div><h2>Heading</h2></div></template>',
 
-    // Test cases ported from ember-template-lint
     '<template><button>Show More</button></template>',
     '<template><button><span>thumbs-up emoji</span>Show More</button></template>',
     '<template><button><div>Show More</div></button></template>',
@@ -32,7 +31,6 @@ ruleTester.run('template-no-heading-inside-button', rule, {
       errors: [{ messageId: 'noHeading' }],
     },
 
-    // Test cases ported from ember-template-lint
     {
       code: '<template><button><h1>Page Title</h1></button></template>',
       output: null,

tests/lib/rules/template-no-inline-styles.js

@@ -8,7 +8,6 @@ const ruleTester = new RuleTester({
 ruleTester.run('template-no-inline-styles', rule, {
   valid: [
     '<template><div class="foo"></div></template>',
-    // Test cases ported from ember-template-lint
     '<template><div></div></template>',
     '<template><span></span></template>',
     '<template><ul class="dummy"></ul></template>',

tests/lib/rules/template-no-input-tagname.js

@@ -9,7 +9,6 @@ const ruleTester = new RuleTester({
 ruleTester.run('template-no-input-tagname', rule, {
   valid: [
     '<template>{{input value=this.foo}}</template>',
-    // Test cases ported from ember-template-lint
     '<template>{{input type="text"}}</template>',
     '<template>{{component "input" type="text"}}</template>',
     '<template>{{yield (component "input" type="text")}}</template>',
@@ -42,7 +41,6 @@ ruleTester.run('template-no-input-tagname', rule, {
       errors: [{ messageId: 'unexpected' }],
     },
 
-    // Test cases ported from ember-template-lint
     {
       code: '<template>{{input tagName="foo"}}</template>',
       output: null,

tests/lib/rules/template-no-invalid-interactive.js

@@ -35,7 +35,6 @@ ruleTester.run('template-no-invalid-interactive', rule, {
       output: null,
     },
 
-    // Test cases ported from ember-template-lint
     '<template><button {{action "foo"}}></button></template>',
     '<template><canvas {{on "mousedown"}}></canvas></template>',
     '<template><div role="button" {{action "foo"}}></div></template>',
@@ -120,7 +119,6 @@ ruleTester.run('template-no-invalid-interactive', rule, {
       ],
     },
 
-    // Test cases ported from ember-template-lint
     {
       code: '<template><div {{on "click" this.actionName}}>...</div></template>',
       output: null,

tests/lib/rules/template-no-invalid-link-text.js

@@ -9,7 +9,6 @@ const ruleTester = new RuleTester({
 ruleTester.run('template-no-invalid-link-text', rule, {
   valid: [
     // Link with component child — content is opaque, can't validate.
-    // Mirrors upstream no-invalid-link-text.js L53-56 ("do not flag when link contains additional dynamic (non-text) children").
     { filename: 'test.gjs', code: '<template><a href="/x"><MyComponent /></a></template>' },
     { filename: 'test.gjs', code: '<template><a href="/x">prefix <MyComponent /></a></template>' },
 

tests/lib/rules/template-no-model-argument-in-route-templates.js

@@ -72,7 +72,6 @@ ruleTester.run('template-no-model-argument-in-route-templates', rule, {
       output: '<template>{{this.model.foo}}</template>',
       errors: [{ messageId: 'noModelArgumentInRouteTemplates' }],
     },
-    // Unknown path defaults to lint (matches upstream).
     {
       code: '<template>{{@model.foo}}</template>',
       output: '<template>{{this.model.foo}}</template>',

tests/lib/rules/template-quotes.js

@@ -28,7 +28,6 @@ const validHbs = [
     code: '<input type=\'checkbox\'> {{hello "test" x="test"}}',
     options: [{ curlies: 'double', html: 'single' }],
   },
-  // `false` as the root config disables the rule (matches upstream).
   {
     code: "{{component \"test\"}} {{hello x='test'}} <input type='checkbox'>",
     options: [false],