CI linting

Signed-off-by: Eric <[email protected]>
opensearch-project · Dec 15, 2023 · 1a9620d · 1a9620d
1 parent f9519e7
commit 1a9620d
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 10 deletions.
diff --git a/.eslintignore b/.eslintignore
@@ -0,0 +1,7 @@
+node_modules
+/data
+/build
+/target
+/.eslintrc.js
+/cypress.config.js
+!.cypress/
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -14,13 +14,37 @@ module.exports = {
     '@elastic/eslint-config-kibana',
     'plugin:@elastic/eui/recommended',
     'plugin:react-hooks/recommended',
+    "eslint:recommended",
+    "plugin:cypress/recommended",
+    "plugin:import/recommended",
+    "prettier"
   ],
   env: {
     'cypress/globals': true,
   },
   plugins: [
     'cypress',
   ],
+  rules: {
+    '@osd/eslint/no-restricted-paths': [
+      'error',
+      {
+        basePath: __dirname,
+        zones: [
+          {
+            target: ['(public|server)/**/*'],
+            from: ['../../packages/**/*','packages/**/*'],
+          },
+        ],
+      },
+    ],
+    // Add cypress specific rules here
+    'cypress/no-assigning-return-values': 'error',
+    'cypress/no-unnecessary-waiting': 'error',
+    'cypress/assertion-before-screenshot': 'warn',
+    'cypress/no-force': 'warn',
+    'cypress/no-async-tests': 'error',
+  },
   overrides: [
     {
       files: ['**/*.{js,ts,tsx}'],
@@ -32,13 +56,8 @@ module.exports = {
             licenses: [LICENSE_HEADER],
           },
         ],
-        // Add cypress specific rules here
-        'cypress/no-assigning-return-values': 'error',
-        'cypress/no-unnecessary-waiting': 'error',
-        'cypress/assertion-before-screenshot': 'warn',
-        'cypress/no-force': 'warn',
-        'cypress/no-async-tests': 'error',
       },
     },
   ],
+  "ignorePatterns": ["**/*.d.ts"]
 };
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -10,7 +10,6 @@ jobs:
     permissions:
       contents: read
       packages: read
-      # To report GitHub Actions status checks
       statuses: write
 
     steps:
@@ -53,11 +52,8 @@ jobs:
       - name: lint code base
         run: |
           cd OpenSearch-Dashboards/plugins/dashboards-observability
-          # Find changed JavaScript/TypeScript files in this PR
-          git remote -v
           git fetch origin main
           CHANGED_FILES=$(git diff --name-only --diff-filter=ACMRTUXB origin/main | grep -E "\.(js|ts|tsx)$")
-          echo "$CHANGED_FILES"
           if [ -n "$CHANGED_FILES" ]; then
             echo "Linting changed files..."
             yarn lint $CHANGED_FILES