Merge pull request #31 from CS222-UIUC/minh/fix_eslint

Fix eslint
CS222-UIUC · Oct 22, 2022 · b46461e · b46461e · github-actions · Oct 22, 2022
2 parents 93b95d7 + 8408bcc
commit b46461e
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/.github/workflows/eslint.yml b/.github/workflows/eslint.yml
@@ -12,6 +12,10 @@ jobs:
         # Make sure the actual branch is checked out when running on pull requests
         ref: ${{ github.head_ref }}
 
+    - name: Install dependencies
+      run: npm install --no-package-lock eslint eslint-plugin-react --legacy-peer-deps
+      working-directory: src/frontend/
+
     - name: Run ESLint
-      run: npx eslint src/. --ext .js,.jsx,.ts,.tsx
+      run: npx eslint --max-warnings 0 $(git ls-files '*.js' '*.jsx' '*.ts' '*.tsx')
       working-directory: src/frontend/
diff --git a/.github/workflows/jest-coverage.yml b/.github/workflows/jest-coverage.yml
@@ -8,12 +8,12 @@ jobs:
       - uses: actions/checkout@v3
 
       - name: Install dependencies
-        run: npm ci
+        run: npm ci --legacy-peer-deps
         working-directory: src/frontend
 
       - name: Run tests
         run: |
-          npx jest --coverage --coverageReporters json-summary
+          npx ts-jest config:init --coverage --coverageReporters json-summary
         working-directory: src/frontend
 
       - name: Jest coverage comment

diff --git a/src/frontend/src/sections/MainPageLeftSection.tsx b/src/frontend/src/sections/MainPageLeftSection.tsx
@@ -1,7 +1,7 @@
 import React from 'react';
 import Searching from '../components/Searching';
 
-export default function LeftSection(props: any) {
+export default function LeftSection() {
    return (
       <div style={{ marginTop: '200px' }}>
          <Searching />