build: Upgrade to Node 24

Author: bydawen

.github/workflows/ci.yml

@@ -12,7 +12,6 @@ jobs:
     strategy:
       matrix:
         node: [20, 24]
-    continue-on-error: ${{ matrix.node == 24 }}
 
     steps:
     - name: Checkout
@@ -22,7 +21,7 @@ jobs:
     - name: Setup Nodejs
       uses: actions/setup-node@v4
       with:
-        node-version: ${{ matrix.node }}
+        node-version-file: '.nvmrc'
     - name: Install dependencies
       run: npm ci
     - name: Validate package-lock.json changes

.nvmrc

@@ -1 +1 @@
-20
+24

package-lock.json

@@ -4,7 +4,7 @@ import { ActionRow, AlertModal, Button } from '@openedx/paragon';
 import { Editor } from '@tinymce/tinymce-react';
 import { useLocation, useParams } from 'react-router-dom';
 // TinyMCE so the global var exists
-// eslint-disable-next-line no-unused-vars,import/no-extraneous-dependencies
+/* eslint-disable-next-line @typescript-eslint/no-unused-vars, import/no-extraneous-dependencies */
 import tinymce from 'tinymce/tinymce';
 
 import { useIntl } from '@edx/frontend-platform/i18n';

src/components/TinyMCEEditor.jsx

@@ -1,4 +1,4 @@
-// eslint-disable-next-line no-unused-vars
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 export default function timeLocale(number, index, totalSec) {
   return [
     ['just now', 'right now'],

src/discussions/common/time-locale.js

@@ -152,6 +152,7 @@ export const useDraftContent = () => {
   };
 
   const removeItem = (draftData, objId) => {
+    /* eslint-disable-next-line @typescript-eslint/naming-convention */
     const { [objId]: _, ...newDraftData } = draftData;
     return newDraftData;
   };

src/discussions/post-comments/data/hooks.js

@@ -0,0 +1,14 @@
+{
+  "extends": "@edx/typescript-config",
+  "compilerOptions": {
+    "noEmit": true,
+    "baseUrl": "./src",
+    "paths": {
+      "*": ["*"]
+    },
+    "jsx": "react-jsx",
+    "allowJs": true
+  },
+  "include": ["*.js", ".eslintrc.js", "src/**/*", "plugins/**/*"],
+  "exclude": ["dist", "node_modules"]
+}