Merge pull request #317 from sbates-idrc/C2LC-575

C2LC-575: Update react-scripts to version 5
codelearncreate · Jan 24, 2022 · 5c0cfa5 · 5c0cfa5
2 parents 6b5e7e1 + fbd2b4e
commit 5c0cfa5
Show file tree

Hide file tree

Showing 7 changed files with 4,263 additions and 9,621 deletions.
diff --git a/.flowcoverage-all b/.flowcoverage-all
diff --git a/.flowcoverage-main-threshold b/.flowcoverage-main-threshold
diff --git a/.flowcoverage-tests-threshold b/.flowcoverage-tests-threshold
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -9,7 +9,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-18.04]
-        node-version: [12.x, 14.x]
+        node-version: [14.x, 16.x]
     runs-on: ${{ matrix.os }}
     env:
       CI: true
@@ -37,14 +37,6 @@ jobs:
         run: npm run build
       - name: Flow
         run: npx flow check
-      - name: Flow Type Coverage (HTML report)
-        run: npx flow-coverage-report --config .flowcoverage-all
-      - name: Archive Flow Type Coverage Report
-        continue-on-error: true
-        uses: actions/upload-artifact@v2
-        with:
-          name: Flow Type Coverage Report
-          path: flow-coverage/
       - name: Test
         run: npm test -- --verbose --coverage
       - name: Archive Test Code Coverage Report
@@ -53,9 +45,5 @@ jobs:
         with:
           name: Test Code Coverage Report
           path: coverage/
-      - name: Flow Type Coverage (main coverage threshold)
-        run: npx flow-coverage-report --config .flowcoverage-main-threshold
-      - name: Flow Type Coverage (test coverage threshold)
-        run: npx flow-coverage-report --config .flowcoverage-tests-threshold
       - name: Lint Documentation
         run: npm run markdownlint
diff --git a/netlify.toml b/netlify.toml
@@ -1,4 +1,4 @@
 [build]
-  environment = { NODE_VERSION = "12" }
+  environment = { NODE_VERSION = "14" }
   command = "npm run netlify-build"
   publish = "build/"
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -7,18 +7,17 @@
         "classnames": "2.3.1",
         "enzyme": "3.11.0",
         "enzyme-adapter-react-16": "1.15.6",
-        "node-sass": "4.14.1",
+        "node-sass": "7.0.1",
         "react": "16.14.0",
         "react-bootstrap": "1.6.4",
         "react-dom": "16.14.0",
         "react-intl": "5.22.0",
-        "react-scripts": "4.0.3",
+        "react-scripts": "5.0.0",
         "tone": "14.7.77",
         "typeface-roboto": "1.1.13"
     },
     "devDependencies": {
         "flow-bin": "0.142.0",
-        "flow-coverage-report": "0.8.0",
         "markdownlint-cli2": "0.3.2"
     },
     "scripts": {
@@ -57,6 +56,7 @@
                     "args": "after-used"
                 }
             ],
+            "no-use-before-define": "off",
             "no-var": "error",
             "prefer-const": "error"
         }