Merge pull request #2544 from johanrd/issue/1488-no-empty-glimmer-declare-class

[BUGFIX]: `no-empty-glimmer-component-classes` false positive on `declare class`

Author: NullVoxPopuli

lib/rules/no-empty-glimmer-component-classes.js

@@ -30,7 +30,7 @@ module.exports = {
   create(context) {
     return {
       ClassDeclaration(node) {
-        if (!isGlimmerComponent(context, node)) {
+        if (node.declare || !isGlimmerComponent(context, node)) {
           return;
         }
 

tests/lib/rules/no-empty-glimmer-component-classes.js

@@ -79,6 +79,15 @@ ruleTester.run('no-empty-glimmer-component-classes', rule, {
       `,
       parser: require.resolve('@typescript-eslint/parser'),
     },
+    // TypeScript declare class should not be flagged
+    {
+      code: `
+        import Component from '@glimmer/component';
+
+        declare class MyComponent extends Component<Sig> {}
+      `,
+      parser: require.resolve('@typescript-eslint/parser'),
+    },
   ],
   invalid: [
     {