Merge pull request #1606 from mightyiam/prettier

prettier

.prettierignore

@@ -0,0 +1 @@
+/CHANGELOG.md

.prettierrc.yml

@@ -0,0 +1,3 @@
+semi: false
+singleQuote: true
+trailingComma: all

.releaserc.yml

@@ -1,9 +1,9 @@
 branches:
   - main
 plugins:
-  - - "@semantic-release/commit-analyzer"
+  - - '@semantic-release/commit-analyzer'
     - preset: conventionalcommits
-  - - "@semantic-release/release-notes-generator"
+  - - '@semantic-release/release-notes-generator'
     - preset: conventionalcommits
       presetConfig:
         types:
@@ -21,10 +21,10 @@ plugins:
             section: Refactoring
           - type: test
             section: Testing
-  - "@semantic-release/changelog"
-  - "@semantic-release/github"
-  - "@semantic-release/npm"
-  - - "@semantic-release/git"
+  - '@semantic-release/changelog'
+  - '@semantic-release/github'
+  - '@semantic-release/npm'
+  - - '@semantic-release/git'
     - assets:
         - CHANGELOG.md
         - package-lock.json

CONTRIBUTING.md

@@ -23,4 +23,3 @@ There's a list of not yet considered rules in [this test file](./src/test/rules-
 Pick one — perhaps the first one — and consider it.
 Is it appropriate for this config?
 If so, [suggest](https://github.com/mightyiam/eslint-config-love/discussions) adding it.
-

README.md

@@ -29,8 +29,8 @@ Here is an example `eslint.config.cjs`.
 module.exports = [
   {
     ...require('eslint-config-love'),
-    files: [ '**/*.js', '**/*.ts' ],
-  }
+    files: ['**/*.js', '**/*.ts'],
+  },
 ]
 ```
 
@@ -41,7 +41,6 @@ Read about the `project` option [here](https://github.com/typescript-eslint/type
 
 There are [some more `parserOptions`](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/parser/README.md#configuration) you may care about.
 
-
 ## Example command line usage:
 
 ```

commitlint.config.js

@@ -1,4 +1,4 @@
 module.exports = {
   extends: ['@commitlint/config-conventional'],
-  'type-enum': ['feat', 'fix', 'style', 'refactor', 'test', 'revert', 'build']
+  'type-enum': ['feat', 'fix', 'style', 'refactor', 'test', 'revert', 'build'],
 }

eslint.config.cjs

@@ -3,24 +3,20 @@ const tseslint = require('typescript-eslint')
 module.exports = [
   require('.'),
   {
-    files: [
-      '**/*.cjs',
-      '**/*.js',
-      '**/*.ts'
-    ],
+    files: ['**/*.cjs', '**/*.js', '**/*.ts'],
     linterOptions: {
-      reportUnusedDisableDirectives: 'error'
-    }
+      reportUnusedDisableDirectives: 'error',
+    },
   },
   {
     files: ['eslint.config.cjs'],
     rules: {
       ...tseslint.configs.disableTypeChecked.rules,
-      '@typescript-eslint/no-var-requires': 'off'
+      '@typescript-eslint/no-var-requires': 'off',
     },
-    languageOptions: { parserOptions: { project: false } }
+    languageOptions: { parserOptions: { project: false } },
   },
   {
-    ignores: ['lib/']
-  }
+    ignores: ['lib/'],
+  },
 ]

package.json

@@ -6,9 +6,11 @@
   "scripts": {
     "compile": "tsc",
     "lint": "eslint .",
+    "format/check": "prettier . --check",
+    "format/fix": "prettier . --write",
     "editorconfig": "editorconfig-checker",
     "unit": "ava",
-    "test": "run-s clean-artifacts editorconfig compile lint unit",
+    "test": "run-s clean-artifacts format/check editorconfig compile lint unit",
     "clean-artifacts": "git clean lib -X --force",
     "is-code-clean": "git diff-index HEAD --quiet",
     "prepublishOnly": "run-s is-code-clean test"
@@ -78,6 +80,7 @@
     "lodash": "4.17.21",
     "npm-package-arg": "11.0.2",
     "npm-run-all": "4.1.5",
+    "prettier": "3.3.2",
     "read-pkg-up": "11.0.0",
     "runtypes": "6.7.0",
     "semantic-release": "24.0.0",

src/fixture.ts

@@ -8,17 +8,19 @@ export type Foo<Bar> = (a: Bar) => Bar
  * https://github.com/mightyiam/eslint-config-love/issues/2
  */
 export default class Zoo {
-  public constructor (private readonly name: string) {}
+  public constructor(private readonly name: string) {}
 
-  public get greeting (): string {
+  public get greeting(): string {
     return `Hello ${this.name}`
   }
 }
 
 /**
  * https://github.com/mightyiam/eslint-config-love/issues/3
  */
-export interface Boo { b_oo: null }
+export interface Boo {
+  b_oo: null
+}
 
 /**
  * ESLint should ignore this `no-undef` violation because that rule is turned off for TypeScript.
@@ -29,8 +31,10 @@ export interface Boo { b_oo: null }
  * https://github.com/mightyiam/eslint-config-love/issues/110
  */
 // Inline callbacks don't need return types:
-setTimeout(() => { console.log() }, 1)
+setTimeout(() => {
+  console.log()
+}, 1)
 
 // The return type is clear from the left side of the assignment:
-const double: ((n: number) => number) = n => n * 2
+const double: (n: number) => number = (n) => n * 2
 ;[1, 2].map(double)