Merge power_linting

.github/workflows/eslint-config.yml

@@ -0,0 +1,44 @@
+name: eslint-config
+
+on:
+  push:
+
+env:
+  PROJECT_DIR: packages/eslint-config
+  NODE_VERSION: '14.18.1'
+  REGISTRY_URL: 'https://registry.npmjs.org'
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    name: Build
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Node ${{ env.NODE_VERSION }}
+        uses: actions/setup-node@v3
+        with: 
+          node-version: ${{ env.NODE_VERSION }}
+      - run: yarn install --frozen-lockfile
+        working-directory: ${{ env.PROJECT_DIR }}
+      - name: Run the build script
+        run: yarn lint
+        working-directory: ${{ env.PROJECT_DIR }}
+
+  release:
+    needs: build
+    runs-on: ubuntu-latest
+    name: Release
+    if: ${{ contains(github.ref, 'refs/tags/v') && contains(github.ref, github.workflow) }}
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Node ${{ env.NODE_VERSION }}
+        uses: actions/setup-node@v3
+        with:
+          node-version: ${{ env.NODE_VERSION }}
+          registry-url: ${{ env.REGISTRY_URL }}
+      - run: yarn install --frozen-lockfile
+        working-directory: ${{ env.PROJECT_DIR }}
+      - run: yarn publish
+        working-directory: ${{ env.PROJECT_DIR }}
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

.github/workflows/rubocop-cobra.yml

@@ -0,0 +1,13 @@
+name: rubocop-cobra
+
+on:
+  push:
+
+jobs:
+  ruby:
+    uses: ./.github/workflows/_ruby-package.yml
+    with:
+      package: ${{ github.workflow }}
+      ruby: '["2.7.4", "3.1.2"]'
+      rails: '["any"]'
+    secrets: inherit

.github/workflows/rubocop-powerhome.yml

@@ -0,0 +1,13 @@
+name: rubocop-powerhome
+
+on:
+  push:
+
+jobs:
+  ruby:
+    uses: ./.github/workflows/_ruby-package.yml
+    with:
+      package: ${{ github.workflow }}
+      ruby: '["2.7.4", "3.1.2"]'
+      rails: '["7.0.3.1","6.1.6.1","6.0.5.1","5.2.8.1"]'
+    secrets: inherit

docs/README.md

@@ -22,6 +22,18 @@ Lumberaxe handles logging output formatting.
 
 Consent provides permission-based authorization.
 
+[rubocop-powerhome](https://github.com/powerhome/power-tools/blob/main/packages/rubocop-powerhome/docs/README.md) 💎
+
+This gem is focused on providing standard rubocop configuration for Power Home Remodeling ruby apps. See more in [`rubocop-powerhome`](../packages/rubocop-powerhome).
+
+[rubocop-cobra](https://github.com/powerhome/power-tools/blob/main/packages/rubocop-cobra/docs/README.md) 💎
+
+This gem is focused on providing Cops to support a healthy cobra app development. See more in [`rubocop-cobra`](../packages/rubocop-cobra).
+
+[@powerhome/eslint-config](https://github.com/powerhome/power-tools/blob/main/packages/eslint-config/docs/README.md)
+
+Shared eslint-config from Power Home Remodeling.
+
 ## Installation 🛠
 
 These packages are all meant to install inside of an application and aren't intended to stand alone; currently, they are all published to [RubyGems](https://rubygems.org/) and you can use standard Bundler methods to install them.

packages/audit_tracker/spec/internal/config/initializers/audit_tracker.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+# rubocop:disable RSpec/FactoryBot/ConsistentParenthesesStyle
 AuditTracker.setup do
   tracker(:user) do
     create :created_by, foreign_key: :created_by_id, class_name: "::Internal::User"
@@ -16,3 +17,4 @@
                                    value: ->(object) { object&.updated_by&.department }
   end
 end
+# rubocop:enable RSpec/FactoryBot/ConsistentParenthesesStyle

packages/eslint-config/.eslintrc.json

@@ -0,0 +1 @@
+{ "extends": "@powerhome/eslint-config/base" }

packages/eslint-config/.gitignore

@@ -0,0 +1,4 @@
+.tmp
+node_modules/
+**/*.d.ts
+**/*.d.ts.map

packages/eslint-config/docs/README.md

@@ -0,0 +1,48 @@
+# @powerhome/eslint-config
+
+Provides eslint-config for Power Home Remodeling apps.
+
+## Installation
+
+Add this line to your application's package.json:
+
+```ruby
+  "devDependencies": {
+    ...
+    "@powerhome/eslint-config": "0.1.0",
+    ...
+  }
+```
+
+And then yarn:
+
+    $ yarn
+
+## Usage
+
+Assuming it's a typescript app, add an `extends` line to your `.eslintrc.json`:
+
+```js
+{
+  ...
+  extends: [
+    ...
+    "@powerhome",
+  ],
+  ...
+}
+```
+
+That's it! You can override the standard configuration after that.
+
+## Flow type apps
+
+For flow-based apps, replace `@powerhome` by `@powerhome/eslint-config/flow`. Note that usage of Flow at Power is not recommended, and all projects should migrate to TypeScript; this set of rules is provided only for transitionary purposes and will be removed in future releases.
+
+## Contributing
+
+Bug reports and pull requests are welcome on GitHub at https://github.com/powerhome/power_linting.
+
+## License
+
+The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

packages/eslint-config/lib/base.js

@@ -0,0 +1,67 @@
+module.exports = {
+  root: true,
+  plugins: ["jsx-control-statements", "react", "react-hooks"],
+  extends: [
+    "eslint:recommended",
+    "plugin:jsx-control-statements/recommended",
+    "plugin:react/recommended",
+    "plugin:react-hooks/recommended",
+  ],
+  parser: "babel-eslint",
+  parserOptions: {
+    ecmaVersion: 6,
+    sourceType: "module",
+    ecmaFeatures: {
+      jsx: true,
+    },
+  },
+  env: {
+    browser: true,
+    es6: true,
+    "jsx-control-statements/jsx-control-statements": true,
+    node: true,
+    mocha: true,
+  },
+  rules: {
+    "jsx-control-statements/jsx-jcs-no-undef": 1,
+    "no-console": 1,
+    "no-restricted-globals": [1, { name: "moment" }],
+    "no-use-before-define": [2, { functions: true, classes: true }],
+    "react/forbid-prop-types": 2,
+    "react/jsx-boolean-value": 2,
+    "react/jsx-closing-bracket-location": 1,
+    "react/jsx-curly-spacing": 2,
+    "react/jsx-handler-names": 2,
+    "react/jsx-indent-props": 2,
+    "react/jsx-key": 2,
+    "react/jsx-max-props-per-line": 2,
+    "react/jsx-no-bind": 0,
+    "react/jsx-no-literals": 1,
+    "react/jsx-no-undef": [2, { allowGlobals: true }],
+    "react/jsx-pascal-case": 2,
+    "react/jsx-sort-props": 2,
+    "react/jsx-wrap-multilines": 2,
+    "react/no-danger": 1,
+    "react/no-did-mount-set-state": 2,
+    "react/no-did-update-set-state": 2,
+    "react/no-multi-comp": 1,
+    "react/no-set-state": 1,
+    "react/prefer-es6-class": 2,
+    "react/prop-types": 1,
+    "react/self-closing-comp": 2,
+    "react/sort-comp": 1,
+  },
+  settings: {
+    react: {
+      version: "detect",
+    },
+  },
+  ignorePatterns: [
+    "**/*.d.ts",
+    "coverage/**/*",
+    "doc/**/*",
+    "node_modules/**/*",
+    "spec/dummy/**/*",
+    "vendor/**/*",
+  ],
+}

packages/eslint-config/lib/flow.js

@@ -0,0 +1,8 @@
+module.exports = {
+  extends: [
+    "@powerhome/eslint-config/base",
+    "plugin:flowtype/recommended"
+  ],
+  plugins: ["flowtype"],
+  ignorePatterns: ["flow-typed/**/*"],
+}

packages/eslint-config/lib/typescript.js

@@ -0,0 +1,24 @@
+module.exports = {
+  extends: ["@powerhome/eslint-config/base"],
+  overrides: [
+    {
+      files: ["**/*.ts", "**/*.tsx"],
+      extends: ["plugin:@typescript-eslint/recommended"],
+      parser: "@typescript-eslint/parser",
+      plugins: ["@typescript-eslint"],
+      rules: {
+        "flowtype/no-types-missing-file-annotation": 0,
+        "jsx-control-statements/jsx-use-if-tag": 0,
+        "no-use-before-define": 0,
+        "@typescript-eslint/no-array-constructor": 0,
+        "@typescript-eslint/explicit-module-boundary-types": 0,
+        "@typescript-eslint/no-var-requires": 0,
+        "@typescript-eslint/no-unused-vars": 1,
+        "@typescript-eslint/no-use-before-define": [
+          2,
+          { functions: true, classes: true },
+        ],
+      },
+    },
+  ],
+}

packages/eslint-config/mkdocs.yml

@@ -0,0 +1,9 @@
+site_name: ESLint Config Powerhome
+site_description: Provides eslint-config for Power Home Remodeling apps
+repo_url: https://github.com/powerhome/power_linting
+edit_uri: edit/main/eslint-config/docs/
+nav:
+  - 'Home': 'README.md'
+
+plugins:
+  - techdocs-core

packages/eslint-config/package.json

@@ -0,0 +1,46 @@
+{
+  "name": "@powerhome/eslint-config",
+  "version": "0.1.0",
+  "license": "MIT",
+  "publishConfig": {
+    "access": "public"
+  },
+  "exports": {
+    "./base": "./lib/base.js",
+    "./flow": "./lib/flow.js",
+    "./typescript": "./lib/typescript.js",
+    ".": "./lib/typescript.js"
+  },
+  "keywords": [
+    "eslint",
+    "eslintconfig"
+  ],
+  "scripts": {
+    "lint": "eslint lib/*"
+  },
+  "peerDependencies": {
+    "eslint": "7.32.0",
+    "eslint-plugin-flowtype": "8.0.3",
+    "eslint-plugin-jsx-control-statements": "3.0.0",
+    "eslint-plugin-react": "7.31.10",
+    "eslint-plugin-react-hooks": "4.6.0",
+    "eslint-webpack-plugin": "3.2.0",
+    "@typescript-eslint/eslint-plugin": "5.41.0",
+    "@typescript-eslint/parser": "5.41.0",
+    "babel-eslint": "8 || 10",
+    "typescript": "4.8.4"
+  },
+  "devDependencies": {
+    "semver": "7.3.8",
+    "eslint": "7.32.0",
+    "eslint-plugin-flowtype": "8.0.3",
+    "eslint-plugin-jsx-control-statements": "3.0.0",
+    "eslint-plugin-react": "7.31.10",
+    "eslint-plugin-react-hooks": "4.6.0",
+    "eslint-webpack-plugin": "3.2.0",
+    "@typescript-eslint/eslint-plugin": "5.41.0",
+    "@typescript-eslint/parser": "5.41.0",
+    "babel-eslint": "10.1.0",
+    "typescript": "4.8.4"
+  }
+}