feat: improve lint tools (#256)

* chore: setup linter and formater tools

fix: add formater to all known files

fix: linter and formatter fixes with some ignored rules

fix: linter fixes

fix: change way to detect tsconfig files

fix: update linter settings

fix: use single qoute always and semicolon just when it's necessary

* fix: add sort imports (#279)

* feat: add support for import sorting plugin

- added import sorting plugin
- also converted from .js file to .json file so typescript compiler
and eslint doesn't complain about commonjs module syntax

* fix: converted all .eslintrc.js to .eslintrc.json

and only use one "root" config so everything is shared

* fix: sort imports lint fix

* fix: import extension error to be disabled in shared elsint config

---------

Co-authored-by: Micah Thomas <95306190+bc-micah@users.noreply.github.com>

.eslintrc.json

@@ -0,0 +1,14 @@
+{
+  "root": true,
+  "extends": ["b2b"],
+  "settings": {
+    "import/resolver": {
+      "typescript": {
+        "project": ["packages/*/tsconfig.json", "apps/*/tsconfig.json"]
+      }
+    }
+  },
+  "parserOptions": {
+    "project": ["packages/*/tsconfig.json", "apps/*/tsconfig.json"]
+  }
+}

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,18 +1,22 @@
 ## What?
+
 Write a description of the changes you're making here, and decisions that you made
 along the way. Provide enough context that the reviewer(s) can understand your
 changes, and the intent behind them. For more details refer to [contributing guideline](https://github.com/bigcommerce/bigcommerce/blob/master/CONTRIBUTING.md#what)
 
 ## Why?
+
 Describe the underlying driver or intent behind this change. Provide some context
 on why it's needed or what is the intended use. For more details refer to [contributing guideline](https://github.com/bigcommerce/bigcommerce/blob/master/CONTRIBUTING.md#why)
 
 ## Testing / Proof
+
 Explain what you did to test and ensure that these changes work the way you expect.
 You must provide evidence in the form of unit tests, screenshots, logs, API
 payloads or other output to show the expected behaviour. For more details refer to [contributing guideline](https://github.com/bigcommerce/bigcommerce/blob/master/CONTRIBUTING.md#testing--proof)
 
 ## How can this change be undone in case of failure?
+
 Provide experiments, feature flags or other mechanisms which used to reduce the risk of changes introduced in this PR. For more details refer to [contributing guideline](https://github.com/bigcommerce/bigcommerce/blob/master/CONTRIBUTING.md#how-can-this-change-be-undone-in-case-of-failure)
 
 ping {suggested reviewers}

.github/workflows/main.yaml

@@ -9,7 +9,6 @@ on:
       - dev
       - main
 
-
 jobs:
   test:
     runs-on: ubuntu-latest

.prettierignore

@@ -0,0 +1,3 @@
+**/dist
+**/out
+**/build

.prettierrc.json

@@ -0,0 +1,4 @@
+{
+  "semi": false,
+  "singleQuote": true
+}

README.md

@@ -6,7 +6,7 @@ This turborepo uses [Yarn](https://classic.yarnpkg.com/lang/en/) as a package
 
 ## Apps and Packages
 
-- `/apps/storefront`: the new BundleB2B storefront aplication,  [React 18](https://reactjs.org/) app using [vite](http://vitejs.dev/) as the building tool.
+- `/apps/storefront`: the new BundleB2B storefront aplication, [React 18](https://reactjs.org/) app using [vite](http://vitejs.dev/) as the building tool.
 
 - `/packages/eslint-config-b3`: the shared eslint config.
 
@@ -26,6 +26,7 @@ This turborepo has the tools and packages already setup:
 - [React Router 6](https://reactrouter.com/) as the frontend router.
 
 ## Setup
+
 1. Run `yarn install`
 2. Copy `apps/storefront/.env-example` and paste it as `.env`
 3. Update the `VITE_STORE_HASH` value on `.env` file

apps/storefront/.eslintrc.json

@@ -0,0 +1,17 @@
+{
+  "extends": ["b2b"],
+  "rules": {
+    // TODO: make a soft update removing one by one
+    "import/no-cycle": 0,
+    "react-hooks/exhaustive-deps": 0,
+    "react/jsx-props-no-spreading": 0,
+    "@typescript-eslint/no-shadow": 0,
+    "no-param-reassign": 0,
+    "@typescript-eslint/no-explicit-any": 0,
+    "no-console": 0,
+    "@typescript-eslint/ban-types": 0,
+    "@typescript-eslint/no-namespace": 0,
+    "react/destructuring-assignment": 0,
+    "@typescript-eslint/no-non-null-assertion": 0
+  }
+}