Remove prettier override so editorconfig is applied (#360)

We had useTabs: true in our prettier config which overrode our .editorconfig to use spaces in json files. I've removed the useTabs in the package.json prettier config so .editorconfig is used instead. I've also reformatted all the code so it's consistent.
preactjs · May 1, 2023 · 6959e94 · 6959e94
1 parent 7973248
commit 6959e94
Show file tree

Hide file tree

Showing 15 changed files with 320 additions and 332 deletions.
diff --git a/.changeset/config.json b/.changeset/config.json
@@ -1,11 +1,11 @@
 {
-	"$schema": "https://unpkg.com/@changesets/[email protected]/schema.json",
-	"changelog": ["@changesets/changelog-github", { "repo": "preactjs/signals" }],
-	"commit": false,
-	"fixed": [],
-	"linked": [],
-	"access": "public",
-	"baseBranch": "main",
-	"updateInternalDependencies": "patch",
-	"ignore": []
+  "$schema": "https://unpkg.com/@changesets/[email protected]/schema.json",
+  "changelog": ["@changesets/changelog-github", { "repo": "preactjs/signals" }],
+  "commit": false,
+  "fixed": [],
+  "linked": [],
+  "access": "public",
+  "baseBranch": "main",
+  "updateInternalDependencies": "patch",
+  "ignore": []
 }
diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json
@@ -1,17 +1,17 @@
 {
-	"name": "preact/signals",
-	"build": {
-		"dockerfile": "Dockerfile"
-	},
-	"customizations": {
-		"vscode": {
-			"settings": {
-				"editor.defaultFormatter": "esbenp.prettier-vscode",
-				"explorer.excludeGitIgnore": true
-			},
-			"extensions": ["dbaeumer.vscode-eslint", "esbenp.prettier-vscode"]
-		}
-	},
-	"postCreateCommand": "pnpm i",
-	"remoteUser": "node"
+  "name": "preact/signals",
+  "build": {
+    "dockerfile": "Dockerfile"
+  },
+  "customizations": {
+    "vscode": {
+      "settings": {
+        "editor.defaultFormatter": "esbenp.prettier-vscode",
+        "explorer.excludeGitIgnore": true
+      },
+      "extensions": ["dbaeumer.vscode-eslint", "esbenp.prettier-vscode"]
+    }
+  },
+  "postCreateCommand": "pnpm i",
+  "remoteUser": "node"
 }
diff --git a/.github/workflows/compressed-size.yml b/.github/workflows/compressed-size.yml
@@ -19,7 +19,6 @@ jobs:
       - name: compressed-size-action
         uses: preactjs/compressed-size-action@v2
         with:
-          pattern: '{packages/*/dist/!(*.module|*.min).{js,mjs},docs/dist/**/*.{js,css}}'
-          build-script: 'ci:build'
+          pattern: "{packages/*/dist/!(*.module|*.min).{js,mjs},docs/dist/**/*.{js,css}}"
+          build-script: "ci:build"
           strip-hash: "[.-](\\w{8,9})\\.(?:js|css)$"
-
diff --git a/README.md b/README.md
@@ -1,4 +1,3 @@
-
 # Signals
 
 Signals is a performant state management library with two primary goals:
@@ -22,17 +21,17 @@ npm install @preact/signals-core
 ```
 
 - [Guide / API](#guide--api)
-	- [`signal(initialValue)`](#signalinitialvalue)
-		- [`signal.peek()`](#signalpeek)
-	- [`computed(fn)`](#computedfn)
-	- [`effect(fn)`](#effectfn)
-	- [`batch(fn)`](#batchfn)
+  - [`signal(initialValue)`](#signalinitialvalue)
+    - [`signal.peek()`](#signalpeek)
+  - [`computed(fn)`](#computedfn)
+  - [`effect(fn)`](#effectfn)
+  - [`batch(fn)`](#batchfn)
 - [Preact Integration](./packages/preact/README.md#preact-integration)
-	- [Hooks](./packages/preact/README.md#hooks)
-	- [Rendering optimizations](./packages/preact/README.md#rendering-optimizations)
-		- [Attribute optimization (experimental)](./packages/preact/README.md#attribute-optimization-experimental)
+  - [Hooks](./packages/preact/README.md#hooks)
+  - [Rendering optimizations](./packages/preact/README.md#rendering-optimizations)
+    - [Attribute optimization (experimental)](./packages/preact/README.md#attribute-optimization-experimental)
 - [React Integration](./packages/react/README.md#react-integration)
-	- [Hooks](./packages/react/README.md#hooks)
+  - [Hooks](./packages/react/README.md#hooks)
 - [License](#license)
 
 ## Guide / API

diff --git a/docs/demos/react/tsconfig.json b/docs/demos/react/tsconfig.json
@@ -1,6 +1,6 @@
 {
   "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "jsxImportSource": "react",
+    "jsxImportSource": "react"
   }
 }
diff --git a/docs/package.json b/docs/package.json
@@ -1,34 +1,34 @@
 {
-	"name": "demo",
-	"private": true,
-	"scripts": {
-		"start": "vite",
-		"build": "vite build",
-		"preview": "vite preview"
-	},
-	"postcss": {
-		"plugins": {
-			"postcss-nesting": {}
-		}
-	},
-	"dependencies": {
-		"preact": "10.9.0",
-		"preact-iso": "^2.3.0",
-		"preact-render-to-string": "^5.2.1",
-		"@preact/signals-core": "workspace:../packages/core",
-		"@preact/signals": "workspace:../packages/preact",
-		"@preact/signals-react": "workspace:../packages/react",
-		"react": "^18.2.0",
-		"react-dom": "^18.2.0"
-	},
-	"devDependencies": {
-		"@babel/core": "^7.18.10",
-		"@preact/preset-vite": "^2.3.0",
-		"@types/react": "^18.0.18",
-		"@types/react-dom": "^18.0.6",
-		"postcss": "^8.4.16",
-		"postcss-nesting": "^10.1.10",
-		"tiny-glob": "^0.2.9",
-		"vite": "^3.0.7"
-	}
+  "name": "demo",
+  "private": true,
+  "scripts": {
+    "start": "vite",
+    "build": "vite build",
+    "preview": "vite preview"
+  },
+  "postcss": {
+    "plugins": {
+      "postcss-nesting": {}
+    }
+  },
+  "dependencies": {
+    "preact": "10.9.0",
+    "preact-iso": "^2.3.0",
+    "preact-render-to-string": "^5.2.1",
+    "@preact/signals-core": "workspace:../packages/core",
+    "@preact/signals": "workspace:../packages/preact",
+    "@preact/signals-react": "workspace:../packages/react",
+    "react": "^18.2.0",
+    "react-dom": "^18.2.0"
+  },
+  "devDependencies": {
+    "@babel/core": "^7.18.10",
+    "@preact/preset-vite": "^2.3.0",
+    "@types/react": "^18.0.18",
+    "@types/react-dom": "^18.0.6",
+    "postcss": "^8.4.16",
+    "postcss-nesting": "^10.1.10",
+    "tiny-glob": "^0.2.9",
+    "vite": "^3.0.7"
+  }
 }
diff --git a/mangle.json b/mangle.json
@@ -5,11 +5,7 @@
   },
   "minify": {
     "mangle": {
-      "reserved": [
-        "useSignal",
-        "useComputed",
-        "useSignalEffect"
-      ],
+      "reserved": ["useSignal", "useComputed", "useSignalEffect"],
       "keep_classnames": true,
       "properties": {
         "regex": "^_[^_]",

diff --git a/package.json b/package.json
@@ -1,99 +1,99 @@
 {
-	"name": "preact-signals",
-	"private": true,
-	"scripts": {
-		"prebuild": "rimraf packages/core/dist/ packages/preact/dist",
-		"build": "pnpm build:core && pnpm build:preact && pnpm build:react",
-		"build:core": "microbundle --raw --cwd packages/core && pnpm postbuild:core",
-		"build:preact": "microbundle --raw --cwd packages/preact && pnpm postbuild:preact",
-		"build:react": "microbundle --raw --cwd packages/react && pnpm postbuild:react",
-		"postbuild:core": "cd packages/core/dist && mv -f index.d.ts signals-core.d.ts",
-		"postbuild:preact": "cd packages/preact/dist && mv -f preact/src/index.d.ts signals.d.ts && rm -dr preact",
-		"postbuild:react": "cd packages/react/dist && mv -f react/src/index.d.ts signals.d.ts && rm -dr react",
-		"postbuild": "node ./scripts/node-13-exports.js",
-		"lint": "eslint 'packages/**/*.{ts,tsx,js,jsx}'",
-		"test": "pnpm test:karma && pnpm test:mocha",
-		"test:minify": "pnpm test:karma:minify && pnpm test:mocha",
-		"test:prod": "pnpm test:karma:prod && pnpm test:mocha:prod",
-		"test:karma": "cross-env COVERAGE=true karma start karma.conf.js --single-run",
-		"test:karma:minify": "cross-env COVERAGE=true MINIFY=true karma start karma.conf.js --single-run",
-		"test:karma:watch": "karma start karma.conf.js --no-single-run",
-		"test:karma:prod": "cross-env MINIFY=true NODE_ENV=production karma start karma.conf.js --single-run",
-		"test:karma:prod:watch": "cross-env NODE_ENV=production karma start karma.conf.js --no-single-run",
-		"test:mocha": "cross-env COVERAGE=true mocha --require packages/react/test/node/setup.js --recursive packages/react/test/node/**.test.tsx",
-		"test:mocha:prod": "cross-env COVERAGE=true NODE_ENV=production mocha --require packages/react/test/node/setup.js --recursive packages/react/test/node/**.test.tsx",
-		"docs:start": "cd docs && pnpm start",
-		"docs:build": "cd docs && pnpm build",
-		"docs:preview": "cd docs && pnpm preview",
-		"ci:build": "pnpm build && pnpm docs:build",
-		"ci:test": "pnpm lint && pnpm test",
-		"release": "pnpm changeset version && pnpm install && git add -A && git commit -m 'Version Packages' && changeset tag && pnpm publish -r"
-	},
-	"authors": [
-		"The Preact Authors (https://github.com/preactjs/signals/contributors)"
-	],
-	"license": "MIT",
-	"devDependencies": {
-		"@babel/core": "^7.19.1",
-		"@babel/plugin-transform-typescript": "^7.19.1",
-		"@babel/preset-env": "^7.19.1",
-		"@babel/preset-react": "^7.18.6",
-		"@babel/preset-typescript": "^7.18.6",
-		"@babel/register": "^7.21.0",
-		"@changesets/changelog-github": "^0.4.6",
-		"@changesets/cli": "^2.24.2",
-		"@types/chai": "^4.3.3",
-		"@types/mocha": "^9.1.1",
-		"@types/node": "^18.6.5",
-		"@types/sinon": "^10.0.13",
-		"@types/sinon-chai": "^3.2.8",
-		"@typescript-eslint/eslint-plugin": "^5.33.0",
-		"@typescript-eslint/parser": "^5.33.0",
-		"babel-plugin-istanbul": "^6.1.1",
-		"babel-plugin-transform-rename-properties": "^0.1.0",
-		"chai": "^4.3.6",
-		"cross-env": "^7.0.3",
-		"errorstacks": "^2.4.0",
-		"esbuild": "^0.14.54",
-		"eslint": "^8.21.0",
-		"eslint-config-prettier": "^8.5.0",
-		"husky": "^8.0.1",
-		"karma": "6.3.16",
-		"karma-chai-sinon": "^0.1.5",
-		"karma-chrome-launcher": "^3.1.1",
-		"karma-coverage": "^2.2.0",
-		"karma-esbuild": "^2.2.5",
-		"karma-mocha": "^2.0.1",
-		"karma-mocha-reporter": "^2.2.5",
-		"karma-sinon": "^1.0.5",
-		"kolorist": "^1.5.1",
-		"lint-staged": "^13.0.3",
-		"microbundle": "^0.15.1",
-		"mocha": "^10.0.0",
-		"prettier": "^2.7.1",
-		"rimraf": "^3.0.2",
-		"sinon": "^14.0.0",
-		"sinon-chai": "^3.7.0",
-		"typescript": "^4.7.4"
-	},
-	"lint-staged": {
-		"**/*.{js,jsx,ts,tsx,yml}": [
-			"prettier --write"
-		]
-	},
-	"husky": {
-		"hooks": {
-			"pre-commit": "lint-staged"
-		}
-	},
-	"prettier": {
-		"useTabs": true,
-		"arrowParens": "avoid"
-	},
-	"pnpm": {
-		"patchedDependencies": {
-			"[email protected]": "patches/[email protected]",
-			"@babel/[email protected]": "patches/@[email protected]"
-		}
-	}
+  "name": "preact-signals",
+  "private": true,
+  "scripts": {
+    "prebuild": "rimraf packages/core/dist/ packages/preact/dist",
+    "build": "pnpm build:core && pnpm build:preact && pnpm build:react",
+    "build:core": "microbundle --raw --cwd packages/core && pnpm postbuild:core",
+    "build:preact": "microbundle --raw --cwd packages/preact && pnpm postbuild:preact",
+    "build:react": "microbundle --raw --cwd packages/react && pnpm postbuild:react",
+    "postbuild:core": "cd packages/core/dist && mv -f index.d.ts signals-core.d.ts",
+    "postbuild:preact": "cd packages/preact/dist && mv -f preact/src/index.d.ts signals.d.ts && rm -dr preact",
+    "postbuild:react": "cd packages/react/dist && mv -f react/src/index.d.ts signals.d.ts && rm -dr react",
+    "postbuild": "node ./scripts/node-13-exports.js",
+    "lint": "eslint 'packages/**/*.{ts,tsx,js,jsx}'",
+    "test": "pnpm test:karma && pnpm test:mocha",
+    "test:minify": "pnpm test:karma:minify && pnpm test:mocha",
+    "test:prod": "pnpm test:karma:prod && pnpm test:mocha:prod",
+    "test:karma": "cross-env COVERAGE=true karma start karma.conf.js --single-run",
+    "test:karma:minify": "cross-env COVERAGE=true MINIFY=true karma start karma.conf.js --single-run",
+    "test:karma:watch": "karma start karma.conf.js --no-single-run",
+    "test:karma:prod": "cross-env MINIFY=true NODE_ENV=production karma start karma.conf.js --single-run",
+    "test:karma:prod:watch": "cross-env NODE_ENV=production karma start karma.conf.js --no-single-run",
+    "test:mocha": "cross-env COVERAGE=true mocha --require packages/react/test/node/setup.js --recursive packages/react/test/node/**.test.tsx",
+    "test:mocha:prod": "cross-env COVERAGE=true NODE_ENV=production mocha --require packages/react/test/node/setup.js --recursive packages/react/test/node/**.test.tsx",
+    "docs:start": "cd docs && pnpm start",
+    "docs:build": "cd docs && pnpm build",
+    "docs:preview": "cd docs && pnpm preview",
+    "ci:build": "pnpm build && pnpm docs:build",
+    "ci:test": "pnpm lint && pnpm test",
+    "release": "pnpm changeset version && pnpm install && git add -A && git commit -m 'Version Packages' && changeset tag && pnpm publish -r",
+    "format": "prettier --ignore-path .gitignore --write '**/*.{js,jsx,ts,tsx,yml,json,md}'"
+  },
+  "authors": [
+    "The Preact Authors (https://github.com/preactjs/signals/contributors)"
+  ],
+  "license": "MIT",
+  "devDependencies": {
+    "@babel/core": "^7.19.1",
+    "@babel/plugin-transform-typescript": "^7.19.1",
+    "@babel/preset-env": "^7.19.1",
+    "@babel/preset-react": "^7.18.6",
+    "@babel/preset-typescript": "^7.18.6",
+    "@babel/register": "^7.21.0",
+    "@changesets/changelog-github": "^0.4.6",
+    "@changesets/cli": "^2.24.2",
+    "@types/chai": "^4.3.3",
+    "@types/mocha": "^9.1.1",
+    "@types/node": "^18.6.5",
+    "@types/sinon": "^10.0.13",
+    "@types/sinon-chai": "^3.2.8",
+    "@typescript-eslint/eslint-plugin": "^5.33.0",
+    "@typescript-eslint/parser": "^5.33.0",
+    "babel-plugin-istanbul": "^6.1.1",
+    "babel-plugin-transform-rename-properties": "^0.1.0",
+    "chai": "^4.3.6",
+    "cross-env": "^7.0.3",
+    "errorstacks": "^2.4.0",
+    "esbuild": "^0.14.54",
+    "eslint": "^8.21.0",
+    "eslint-config-prettier": "^8.5.0",
+    "husky": "^8.0.1",
+    "karma": "6.3.16",
+    "karma-chai-sinon": "^0.1.5",
+    "karma-chrome-launcher": "^3.1.1",
+    "karma-coverage": "^2.2.0",
+    "karma-esbuild": "^2.2.5",
+    "karma-mocha": "^2.0.1",
+    "karma-mocha-reporter": "^2.2.5",
+    "karma-sinon": "^1.0.5",
+    "kolorist": "^1.5.1",
+    "lint-staged": "^13.0.3",
+    "microbundle": "^0.15.1",
+    "mocha": "^10.0.0",
+    "prettier": "^2.7.1",
+    "rimraf": "^3.0.2",
+    "sinon": "^14.0.0",
+    "sinon-chai": "^3.7.0",
+    "typescript": "^4.7.4"
+  },
+  "lint-staged": {
+    "**/*.{js,jsx,ts,tsx,yml,json,md}": [
+      "prettier --write"
+    ]
+  },
+  "husky": {
+    "hooks": {
+      "pre-commit": "lint-staged"
+    }
+  },
+  "prettier": {
+    "arrowParens": "avoid"
+  },
+  "pnpm": {
+    "patchedDependencies": {
+      "[email protected]": "patches/[email protected]",
+      "@babel/[email protected]": "patches/@[email protected]"
+    }
+  }
 }