Fix Flow types for class static blocks

Summary: Original Author: [email protected] Original Git: ce1c4dd Original Reviewed By: avp Original Revision: D66737666 A type needed to be updated in `ClassStaticBlockScope`. Also ordered the types in `type Statement` alphabetically since the rest are. Reviewed By: avp Differential Revision: D66832592 fbshipit-source-id: 36209df85538c8a5a58ea203f8809dcb5b8e7355
facebook · Dec 5, 2024 · 4a42dbf · 4a42dbf
1 parent 48653e7
commit 4a42dbf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/tools/hermes-parser/js/hermes-eslint/src/scope-manager/scope/ClassStaticBlockScope.js b/tools/hermes-parser/js/hermes-eslint/src/scope-manager/scope/ClassStaticBlockScope.js
@@ -10,7 +10,7 @@
 
 'use strict';
 
-import type {Expression} from 'hermes-estree';
+import type {StaticBlock} from 'hermes-estree';
 import type {ClassScope} from './ClassScope';
 import type {ScopeManager} from '../ScopeManager';
 
@@ -19,7 +19,7 @@ import {ScopeType} from './ScopeType';
 
 class ClassStaticBlockScope extends ScopeBase<
   typeof ScopeType.ClassStaticBlock,
-  Expression,
+  StaticBlock,
   ClassScope,
 > {
   declare +type: typeof ScopeType.ClassStaticBlock;

diff --git a/tools/hermes-parser/js/hermes-estree/src/types.js b/tools/hermes-parser/js/hermes-estree/src/types.js
@@ -232,7 +232,6 @@ export type AFunction =
 
 export type Statement =
   | BlockStatement
-  | StaticBlock
   | BreakStatement
   | ClassDeclaration
   | ComponentDeclaration
@@ -263,6 +262,7 @@ export type Statement =
   | LabeledStatement
   | OpaqueType
   | ReturnStatement
+  | StaticBlock
   | SwitchStatement
   | ThrowStatement
   | TryStatement