feat(prettier)!: use tabs instead of spaces (#42)

Resolves #34
mheob · Sep 16, 2022 · cf8ba94 · cf8ba94
1 parent 7aad5cc
commit cf8ba94
Show file tree

Hide file tree

Showing 25 changed files with 401 additions and 394 deletions.
diff --git a/.changeset/lemon-cheetahs-look.md b/.changeset/lemon-cheetahs-look.md
@@ -0,0 +1,6 @@
+---
+'@mheob/eslint-config': patch
+'@mheob/prettier-config': patch
+---
+
+Format all files to the new tabs instead of spaces behaviour
diff --git a/.changeset/light-llamas-smile.md b/.changeset/light-llamas-smile.md
@@ -0,0 +1,5 @@
+---
+'@mheob/prettier-config': major
+---
+
+Switch from spaces to tabs as indentation
diff --git a/.editorconfig b/.editorconfig
@@ -4,7 +4,7 @@ root = true
 [*]
 charset = utf-8
 end_of_line = lf
-indent_style = space
+indent_style = tab
 indent_size = 2
 insert_final_newline = true
 trim_trailing_whitespace = true

diff --git a/.eslintrc.cjs b/.eslintrc.cjs
@@ -1,5 +1,5 @@
 /** @type {import('eslint').ESLint.ConfigData} */
 module.exports = {
-  root: true,
-  extends: ['@mheob/eslint-config'],
+	root: true,
+	extends: ['@mheob/eslint-config'],
 };
diff --git a/.github/renovate.json b/.github/renovate.json
@@ -1,25 +1,25 @@
 {
-  "$schema": "https://docs.renovatebot.com/renovate-schema.json",
-  "extends": ["config:base"],
-  "labels": ["dependencies"],
-  "rangeStrategy": "replace",
-  "separateMajorMinor": true,
-  "dependencyDashboard": true,
-  "dependencyDashboardAutoclose": false,
-  "major": {
-    "dependencyDashboardApproval": true
-  },
-  "stabilityDays": 3,
-  "prCreation": "not-pending",
-  "prConcurrentLimit": 5,
-  "timezone": "Europe/Berlin",
-  "schedule": ["before 5am on Wednesday"],
-  "packageRules": [
-    {
-      "matchPackagePatterns": ["*"],
-      "matchUpdateTypes": ["minor", "patch"],
-      "groupName": "all non-major dependencies",
-      "groupSlug": "all-minor-patch"
-    }
-  ]
+	"$schema": "https://docs.renovatebot.com/renovate-schema.json",
+	"extends": ["config:base"],
+	"labels": ["dependencies"],
+	"rangeStrategy": "replace",
+	"separateMajorMinor": true,
+	"dependencyDashboard": true,
+	"dependencyDashboardAutoclose": false,
+	"major": {
+		"dependencyDashboardApproval": true
+	},
+	"stabilityDays": 3,
+	"prCreation": "not-pending",
+	"prConcurrentLimit": 5,
+	"timezone": "Europe/Berlin",
+	"schedule": ["before 5am on Wednesday"],
+	"packageRules": [
+		{
+			"matchPackagePatterns": ["*"],
+			"matchUpdateTypes": ["minor", "patch"],
+			"groupName": "all non-major dependencies",
+			"groupSlug": "all-minor-patch"
+		}
+	]
 }
diff --git a/.husky/post-merge b/.husky/post-merge
@@ -6,9 +6,9 @@
 changedFiles="$(git diff-tree -r --name-only --no-commit-id ORIG_HEAD HEAD)"
 
 hasChanged() {
-  echo "$changedFiles" | grep -q "$1"
+	echo "$changedFiles" | grep -q "$1"
 }
 
 if hasChanged 'pnpm-lock.yaml'; then
-  pnpm install
+	pnpm install
 fi
diff --git a/.prettierrc.cjs b/.prettierrc.cjs
@@ -1,3 +1,3 @@
 module.exports = {
-  ...require('@mheob/prettier-config'),
+	...require('@mheob/prettier-config'),
 };
diff --git a/package.json b/package.json
@@ -1,51 +1,50 @@
 {
-  "name": "@mheob/config",
-  "version": "0.0.0",
-  "private": true,
-  "license": "MIT",
-  "workspaces": [
-    "packages/*"
-  ],
-  "scripts": {
-    "changeset": "changeset",
-    "clean": "turbo run clean",
-    "clean:root": "rm -rf .turbo && rm -rf node_modules",
-    "format": "prettier --write \"**/*.{cjs,js,jsx,ts,tsx,json,md,mdx,yml}\"",
-    "lint": "turbo run lint",
-    "prepare": "ts-node ./scripts/prepare.ts",
-    "release": "changeset publish",
-    "sort-package-json": "pnpm dlx sort-package-json && turbo run sort-package-json",
-    "version-packages": "changeset version"
-  },
-  "commitlint": {
-    "extends": [
-      "@commitlint/config-conventional"
-    ]
-  },
-  "lint-staged": {
-    "*.{cjs,js,jsx,ts,tsx}": "eslint --fix",
-    "!(pnpm-)*.{cjs,js,jsx,ts,tsx,json,md,mdx,yml,yaml}": "pnpm exec prettier --write",
-    "**/package.json": "pnpm dlx sort-package-json"
-  },
-  "devDependencies": {
-    "@changesets/cli": "^2.24.4",
-    "@commitlint/cli": "^17.1.2",
-    "@commitlint/config-conventional": "^17.1.0",
-    "@mheob/eslint-config": "workspace:*",
-    "@mheob/prettier-config": "workspace:*",
-    "@types/node": "^18.7.18",
-    "eslint": "^8.23.1",
-    "husky": "^8.0.0",
-    "is-ci": "^3.0.1",
-    "lint-staged": "^13.0.3",
-    "prettier": "^2.7.1",
-    "ts-node": "^10.9.1",
-    "turbo": "^1.4.6",
-    "typescript": "^4.8.3"
-  },
-  "packageManager": "pnpm@7.11.0",
-  "engines": {
-    "node": ">=16.0.0",
-    "npm": ">=8.0.0"
-  }
+	"name": "@mheob/config",
+	"version": "0.0.0",
+	"private": true,
+	"license": "MIT",
+	"workspaces": [
+		"packages/*"
+	],
+	"scripts": {
+		"changeset": "changeset",
+		"clean": "turbo run clean",
+		"clean:root": "rm -rf .turbo && rm -rf node_modules",
+		"format": "prettier --write \"**/*.{cjs,js,jsx,ts,tsx,json,md,mdx,yml}\"",
+		"lint": "turbo run lint",
+		"prepare": "ts-node ./scripts/prepare.ts",
+		"release": "changeset publish",
+		"sort-package-json": "pnpm dlx sort-package-json && turbo run sort-package-json",
+		"version-packages": "changeset version"
+	},
+	"commitlint": {
+		"extends": [
+			"@commitlint/config-conventional"
+		]
+	},
+	"lint-staged": {
+		"*.{cjs,js,jsx,ts,tsx}": "eslint --fix",
+		"!(pnpm-)*.{cjs,js,jsx,ts,tsx,json,md,mdx,yml,yaml}": "pnpm exec prettier --write",
+		"**/package.json": "pnpm dlx sort-package-json"
+	},
+	"devDependencies": {
+		"@changesets/cli": "^2.24.4",
+		"@commitlint/cli": "^17.1.2",
+		"@commitlint/config-conventional": "^17.1.0",
+		"@mheob/eslint-config": "workspace:*",
+		"@mheob/prettier-config": "workspace:*",
+		"@types/node": "^18.7.18",
+		"eslint": "^8.23.1",
+		"husky": "^8.0.0",
+		"lint-staged": "^13.0.3",
+		"prettier": "^2.7.1",
+		"ts-node": "^10.9.1",
+		"turbo": "^1.4.6",
+		"typescript": "^4.8.3"
+	},
+	"packageManager": "pnpm@7.11.0",
+	"engines": {
+		"node": ">=16.0.0",
+		"npm": ">=8.0.0"
+	}
 }
diff --git a/packages/eslint-config/.eslintrc.cjs b/packages/eslint-config/.eslintrc.cjs
@@ -1,8 +1,8 @@
 /** @type {import('eslint').ESLint.ConfigData} */
 module.exports = {
-  extends: ['./base.js'],
-  env: { node: true },
-  rules: {
-    'unicorn/prefer-module': 'off',
-  },
+	extends: ['./base.js'],
+	env: { node: true },
+	rules: {
+		'unicorn/prefer-module': 'off',
+	},
 };
diff --git a/packages/eslint-config/README.md b/packages/eslint-config/README.md
@@ -29,8 +29,8 @@ pnpm add -D @mheob/eslint-config
 ```js
 // .eslintrc.cjs
 module.exports = {
-  root: true, // optional
-  extends: ['@mheob/eslint-config'],
+	root: true, // optional
+	extends: ['@mheob/eslint-config'],
 };
 ```
 
@@ -39,8 +39,8 @@ module.exports = {
 ```js
 // .eslintrc.cjs
 module.exports = {
-  root: true, // optional
-  extends: ['@mheob/eslint-config/react'],
+	root: true, // optional
+	extends: ['@mheob/eslint-config/react'],
 };
 ```
 
@@ -49,8 +49,8 @@ module.exports = {
 ```js
 // .eslintrc.cjs
 module.exports = {
-  root: true, // optional
-  extends: ['@mheob/eslint-config/next'],
+	root: true, // optional
+	extends: ['@mheob/eslint-config/next'],
 };
 ```
 
@@ -59,12 +59,12 @@ If you need to override some rules you can do it this way:
 ```js
 // .eslintrc.cjs
 module.exports = {
-  root: true, // optional
-  extends: ['@mheob/eslint-config'],
-  rules: {
-    'prettier/prettier': 'off',
-    'no-console': 'warn',
-  },
+	root: true, // optional
+	extends: ['@mheob/eslint-config'],
+	rules: {
+		'prettier/prettier': 'off',
+		'no-console': 'warn',
+	},
 };
 ```
 
@@ -73,17 +73,17 @@ Or override rules specified for some files:
 ```js
 // .eslintrc.cjs
 module.exports = {
-  root: true, // optional
-  extends: ['@mheob/eslint-config'],
-  overrides: [
-    {
-      files: ['*.jsx', '*.tsx'],
-      settings: { react: { version: 'detect' } },
-      rules: {
-        'react/jsx-no-useless-fragment': 'warn',
-        'react/react-in-jsx-scope': 'off',
-      },
-    },
-  ],
+	root: true, // optional
+	extends: ['@mheob/eslint-config'],
+	overrides: [
+		{
+			files: ['*.jsx', '*.tsx'],
+			settings: { react: { version: 'detect' } },
+			rules: {
+				'react/jsx-no-useless-fragment': 'warn',
+				'react/react-in-jsx-scope': 'off',
+			},
+		},
+	],
 };
 ```
diff --git a/packages/eslint-config/base.js b/packages/eslint-config/base.js
@@ -3,50 +3,50 @@
  * @type {import('eslint').ESLint.ConfigData}
  */
 module.exports = {
-  plugins: ['@typescript-eslint'],
-  extends: ['eslint:recommended', 'plugin:unicorn/recommended', 'plugin:prettier/recommended'],
-  rules: {
-    'prettier/prettier': 'warn',
-    'no-console': ['warn', { allow: ['warn', 'error'] }],
-    'no-implicit-coercion': ['error', { allow: ['!!', '+', '~'] }],
-    'no-unused-vars': ['warn', { argsIgnorePattern: '^_', varsIgnorePattern: '^_' }],
-    'unicorn/import-index': 'error',
-    'unicorn/prevent-abbreviations': [
-      'error',
-      {
-        replacements: {
-          args: false,
-          doc: false,
-          env: false,
-          props: false,
-        },
-      },
-    ],
-  },
-  overrides: [
-    {
-      files: ['*.cjs'],
-      env: { node: true },
-    },
-    {
-      files: ['*.js', '*.ts'],
-      rules: { 'unicorn/filename-case': ['error', { case: 'camelCase' }] },
-    },
-    {
-      files: ['*.ts', '*.tsx'],
-      extends: ['plugin:@typescript-eslint/recommended'],
-      rules: {
-        '@typescript-eslint/consistent-type-imports': 'error',
-        '@typescript-eslint/explicit-function-return-type': [
-          'warn',
-          { allowExpressions: true, allowTypedFunctionExpressions: true },
-        ],
-        '@typescript-eslint/no-unused-vars': [
-          'warn',
-          { argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
-        ],
-        '@typescript-eslint/quotes': ['warn', 'single', { avoidEscape: true }],
-      },
-    },
-  ],
+	plugins: ['@typescript-eslint'],
+	extends: ['eslint:recommended', 'plugin:unicorn/recommended', 'plugin:prettier/recommended'],
+	rules: {
+		'prettier/prettier': 'warn',
+		'no-console': ['warn', { allow: ['warn', 'error'] }],
+		'no-implicit-coercion': ['error', { allow: ['!!', '+', '~'] }],
+		'no-unused-vars': ['warn', { argsIgnorePattern: '^_', varsIgnorePattern: '^_' }],
+		'unicorn/import-index': 'error',
+		'unicorn/prevent-abbreviations': [
+			'error',
+			{
+				replacements: {
+					args: false,
+					doc: false,
+					env: false,
+					props: false,
+				},
+			},
+		],
+	},
+	overrides: [
+		{
+			files: ['*.cjs'],
+			env: { node: true },
+		},
+		{
+			files: ['*.js', '*.ts'],
+			rules: { 'unicorn/filename-case': ['error', { case: 'camelCase' }] },
+		},
+		{
+			files: ['*.ts', '*.tsx'],
+			extends: ['plugin:@typescript-eslint/recommended'],
+			rules: {
+				'@typescript-eslint/consistent-type-imports': 'error',
+				'@typescript-eslint/explicit-function-return-type': [
+					'warn',
+					{ allowExpressions: true, allowTypedFunctionExpressions: true },
+				],
+				'@typescript-eslint/no-unused-vars': [
+					'warn',
+					{ argsIgnorePattern: '^_', varsIgnorePattern: '^_' },
+				],
+				'@typescript-eslint/quotes': ['warn', 'single', { avoidEscape: true }],
+			},
+		},
+	],
 };
diff --git a/packages/eslint-config/next.js b/packages/eslint-config/next.js
@@ -3,5 +3,5 @@
  * @type {import('eslint').ESLint.ConfigData}
  */
 module.exports = {
-  extends: ['./react', 'next/core-web-vitals'],
+	extends: ['./react', 'next/core-web-vitals'],
 };