Merge pull request #2960 from exadel-inc/epic/linting

epic: linting packages rework + stylelint config package + initiating classic monorepo structure

.codeclimate.yml

@@ -25,22 +25,22 @@ checks:
   return-statements:
     config:
       threshold: 6
-#  similar-code:
-#    config:
-#      threshold: 45
-#  identical-code:
-#    config:
-#      threshold: 45
+  similar-code:
+    config:
+      threshold: 60
 
 exclude_patterns:
   - "config/"
   - "dist/"
-  - "features/"
   - "**/node_modules/"
   - "**/spec/"
   - "**/test/"
   - "**/tests/"
   - "**/vendor/"
   - "**/*.d.ts"
   - "**/*.test.ts"
-  - "e2e/setup/**/*.ts"
+  - "e2e/**/*.ts"
+
+  - "packages/eslint-config"
+  - "packages/eslint-plugin"
+  - "packages/stylelint-config"

.commitlintrc.yml

@@ -65,6 +65,7 @@ rules:
       - deps-dev
       - eslint-plugin
       - eslint-config
+      - stylelint-config
       - e2e
       - ci
       # Deprecated scopes:

.github/workflows/validate.yml

@@ -67,4 +67,4 @@ jobs:
       - name: Install NPM Dependencies
         run: npm ci
       - name: Run ESLint Plugin Tests
-        run: npm run test -w eslint-plugin
+        run: npm run test -w packages/eslint-plugin

.gitignore

@@ -11,9 +11,9 @@ node_modules
 # Build output
 *.tgz
 target
-site/dist
 
-eslint-plugin/dist
+site/dist
+e2e/.diff
 
 # Generated surces
 /modules
@@ -29,4 +29,3 @@ npm-debug.log
 # Jest & Sonar
 /.report
 /.scannerwork
-/e2e/.diff

.releaserc.yml

@@ -68,8 +68,7 @@ plugins:
         - package.json
         - package-lock.json
         - site/package.json
-        - eslint-config/package.json
-        - eslint-plugin/package.json
+        - packages/*/package.json
       message: "chore(release): ${nextRelease.version} \n\n${nextRelease.notes}"
 
   - - "@semantic-release/github"

.stylelintrc.yml

@@ -1,16 +1,13 @@
 plugins:
-  - stylelint-prettier
   # Custom plugin to disallow absolute import paths
   - ./build/import.stylelint.js
 # Use LESS preprocessor
 customSyntax: postcss-less
 
-rules:
-  # Prettier integration
-  prettier/prettier:
-    - true
-    - severity: warning
+extends:
+  - '@exadel/stylelint-config-esl'
 
+rules:
   # Disallow unknown at-rules
   at-rule-no-unknown: true
   # Warn about empty selectors
@@ -73,7 +70,7 @@ rules:
 
 overrides:
   # Demo site pages sources
-  - files: site/**/*.{css,less}
+  - files: ['site/**/*.{css,less}']
     rules:
       # Warn about duplicate properties within declaration blocks
       declaration-block-no-duplicate-properties:

docs/DEVELOPMENT.md

@@ -61,7 +61,7 @@ ESL codebase is written using TypeScript and LESS CSS-preprocessor.
 
 ESL uses the following tools to keep codebase quality
 - ESLint to lint scripts
-- Own ESLint shared configuration (see [eslint-config](../eslint-config) sub-package)
+- Own ESLint shared configuration (see [eslint-config](../packages/eslint-config) sub-package)
 - StyleLint to lint styles
 - Jest to run unit tests
 - CommitLint to check commit message format
@@ -112,15 +112,15 @@ ESL project consists of the following directories:
   - [🔧 tsconfig.json](../site/tsconfig.json) - TS config for demo pages scripts
   - [🔧 webpack.config.js](../site/webpack.config.js) - webpack build file for demo pages
 
-
-- [📁 eslint-plugin](../eslint-plugin) - sub-package root for ESL ESLint plugin 
-  - [📁 src](../eslint-plugin/src) - ESLint plugin sources
-  - [📁 test](../eslint-plugin/test) - ESLint plugin tests
-
-- [📁 eslint-config](../eslint-config) - sub-package root for ESL ESLint shared configuration
-  - [📁 rules](../eslint-config/rules) - ESLint shared configuration rule sets
-  - [📄 index.js](../eslint-config/index.js) - ESLint shared configuration main file
-
+- [📁 packages](../packages) - ESL monorepo sub-packages
+  - [📁 eslint-plugin](../packages/eslint-plugin) - sub-package root for ESL ESLint plugin 
+    - [📁 src](../packages/eslint-plugin/src) - ESLint plugin sources
+    - [📁 test](../packages/eslint-plugin/test) - ESLint plugin tests
+  - [📁 eslint-config](../packages/eslint-config) - sub-package root for ESL ESLint shared configuration
+    - [📁 rules](../packages/eslint-config/rules) - ESLint shared configuration rule sets
+    - [📄 index.js](../packages/eslint-config/index.js) - ESLint shared configuration main file
+  - [📁 stylelint-config](../packages/stylelint-config) - sub-package root for ESL StyleLint shared configuration
+    - [📄 index.js](../packages/stylelint-config/index.js) - StyleLint shared configuration main file
 
 - [📁 build](../build) - library common build scripts
 - [📁 linting](../linting) - ES Lint rule-sets

eslint.config.ignore.js → eslint.config.ignore.mjs

@@ -1,4 +1,4 @@
-module.exports = [
+export default [
   {
     ignores: [
       // Common configuration
@@ -15,13 +15,13 @@ module.exports = [
       '/polyfills/**',
       // Site output
       '/site/dist/**',
-      // ESL ESLint Plugin output
-      '/eslint-plugin/dist/**',
+      // ESL ESLint Plugin files
+      '/packages/eslint-plugin/**',
       // ESL ESLint Config files
-      '/eslint-config/**',
+      '/packages/eslint-config/**',
       // E2E tests commons
       'e2e/reporters/**',
       'e2e/transformer/**',
     ]
   }
-]
+];