Fix the Prettier CI check (#96)

* Fix the prettier-check in CI

* Run prettier during bump-version

* Run prettier

Author: olafurpg

CHANGELOG.md

@@ -1,13 +1,15 @@
 ## LSIF Typescript
 
 ### v0.1.16 (2022/04/26 02:07 +00:00)
+
 - [#90](https://github.com/sourcegraph/lsif-typescript/pull/90) Update `--infer-tsconfig` to skip the `allowJs` option when there exist TS files (#90) (@olafurpg)
 - [#89](https://github.com/sourcegraph/lsif-typescript/pull/89) Update dependency @sourcegraph/eslint-config to v0.27.1 (#89) (@renovate[bot])
 - [#86](https://github.com/sourcegraph/lsif-typescript/pull/86) Correctly emit definitions for type aliases. (#86) (@varungandhi-src)
 - [#84](https://github.com/sourcegraph/lsif-typescript/pull/84) fix: Emit definition for class declarations. (#84) (@varungandhi-src)
 - [#83](https://github.com/sourcegraph/lsif-typescript/pull/83) fix: Use allowJS in --infer-tsconfig. (#83) (@varungandhi-src)
 
 ### v0.1.15 (2022/04/13 19:10 +00:00)
+
 - [#80](https://github.com/sourcegraph/lsif-typescript/pull/80) Update ChangeLog and bump version for releasing 0.1.15. (#80) (@varungandhi-src)
 - [#77](https://github.com/sourcegraph/lsif-typescript/pull/77) ci: Fix bot name/email. (#77) (@varungandhi-src)
 - [#74](https://github.com/sourcegraph/lsif-typescript/pull/74) Add support for accessors. (#74) (@varungandhi-src)
@@ -21,21 +23,25 @@
 - [#66](https://github.com/sourcegraph/lsif-typescript/pull/66) chore: Add generated .lsif-typed files to .gitignore. (#66) (@varungandhi-src)
 
 ### v0.1.14 (2022/03/21 14:53 +00:00)
+
 - [#50](https://github.com/sourcegraph/lsif-typescript/pull/50) Merge pull request #50 from sourcegraph/duplicate-definitions (@olafurpg)
 - [#49](https://github.com/sourcegraph/lsif-typescript/pull/49) Merge pull request #49 from sourcegraph/commander (@olafurpg)
 - [#47](https://github.com/sourcegraph/lsif-typescript/pull/47) Merge pull request #47 from sourcegraph/cross-project (@olafurpg)
 - [#28](https://github.com/sourcegraph/lsif-typescript/pull/28) Merge pull request #28 from sourcegraph/renovate/node-17.x (@renovate-bot)
 
 ### v0.1.11 (2022/03/16 13:02 +00:00)
+
 - [#40](https://github.com/sourcegraph/lsif-typescript/pull/40) Merge pull request #40 from sourcegraph/allowjs (@olafurpg)
 - [#39](https://github.com/sourcegraph/lsif-typescript/pull/39) Merge pull request #39 from sourcegraph/progress-bar (@olafurpg)
 - [#36](https://github.com/sourcegraph/lsif-typescript/pull/36) Merge pull request #36 from sourcegraph/sg-upload (@olafurpg)
 
 ### v0.1.5 (2022/03/07 10:57 +00:00)
+
 - [#26](https://github.com/sourcegraph/lsif-typescript/pull/26) Merge pull request #26 from sourcegraph/renovate/major-react-monorepo (@renovate-bot)
 - [#22](https://github.com/sourcegraph/lsif-typescript/pull/22) Merge pull request #22 from olafurpg/react (@olafurpg)
 
 ### v0.1.0 (2022/02/25 15:43 +00:00)
+
 - [#6](https://github.com/sourcegraph/lsif-typescript/pull/6) Merge pull request #6 from olafurpg/release (@olafurpg)
 - [#4](https://github.com/sourcegraph/lsif-typescript/pull/4) Merge pull request #4 from sourcegraph/add-pr-auditor-and-test-plan (@sourcegraph)
-- [#1](https://github.com/sourcegraph/lsif-typescript/pull/1) Merge pull request #1 from olafurpg/new-start (@olafurpg)
+- [#1](https://github.com/sourcegraph/lsif-typescript/pull/1) Merge pull request #1 from olafurpg/new-start (@olafurpg)

dev/bump-version

@@ -16,6 +16,7 @@ echo "Updating package.json."
 yarn version --no-git-tag-version --new-version $1
 echo "Updating CHANGELOG.md."
 npx github-changes --only-pulls --branch main --owner sourcegraph --repository lsif-typescript --title "LSIF Typescript" --tag-name "v$1" --token "$GITHUB_TOKEN"
+yarn run prettier
 git add .
 TITLE="Update ChangeLog and bump version for releasing $1."
 git commit -m "$TITLE"

package.json

@@ -7,7 +7,7 @@
   "main": "./dist/src/main.js",
   "scripts": {
     "prettier": "prettier --write --list-different '**/*.{ts,js?(on),md,yml}'",
-    "prettier-check": "npm run prettier --write=false",
+    "prettier-check": "prettier --check '**/*.{ts,js?(on),md,yml}'",
     "tslint": "tslint -p tsconfig.json --format stylish",
     "eslint": "eslint --cache '**/*.ts?(x)'",
     "build": "node ./node_modules/typescript/bin/tsc -b .",

snapshots/input/multi-project/packages/a/tsconfig.json

@@ -8,5 +8,5 @@
       "@example/a/src/*": ["src/*"]
     }
   },
-  "include": ["src/*"],
+  "include": ["src/*"]
 }

snapshots/input/multi-project/packages/b/tsconfig.json

@@ -4,7 +4,7 @@
     "rootDir": ".",
     "baseUrl": "./src",
     "sourceRoot": "src",
-    "outDir": "dist",
+    "outDir": "dist"
   },
   "include": ["src/*"],
   "references": [{ "path": "../a" }]

snapshots/input/pure-js/src/main.js

@@ -1,46 +1,49 @@
 function fib(n) {
   if (n <= 1) {
-    return 0;
+    return 0
   }
-  return fib(n - 1) + fib(n - 2);
+  return fib(n - 1) + fib(n - 2)
 }
 
 function print_fib(a) {
-  console.log(fib(a));
+  console.log(fib(a))
 }
 
-var y = "Hello";
+var y = 'Hello'
 function capture() {
-  return y;
+  return y
 }
-const capture_lambda = () => { return y; };
-
-for (var i = 0; i <= 10; i++) {
+const capture_lambda = () => {
+  return y
 }
 
+for (var i = 0; i <= 10; i++) {}
+
 for (const x of [1, 2, 3]) {
 }
 
-var a = 0;
-var a = 1;
-print_fib(a);
+var a = 0
+var a = 1
+print_fib(a)
 
-function forever() { return forever(); }
+function forever() {
+  return forever()
+}
 
 function use_before_def() {
-  print_fib(n);
-  var n = 10;
+  print_fib(n)
+  var n = 10
 
   if (forever()) {
-    var m = 10;
+    var m = 10
   }
-  print_fib(m);
+  print_fib(m)
 }
 
 function var_function_scope() {
-  var k = 0;
+  var k = 0
   if (forever()) {
-    var k = 1;
+    var k = 1
   }
-  print_fib(k);
+  print_fib(k)
 }

snapshots/input/syntax/src/accessors.ts

@@ -1,10 +1,10 @@
 class C {
-  _length: number = 0;
+  _length: number = 0
   get length(): number {
-    return this._length;
+    return this._length
   }
   set length(value: number) {
-    this._length = value;
+    this._length = value
   }
 
   _capacity: number = 0
@@ -14,24 +14,24 @@ class C {
 }
 
 export class D {
-    _length: number = 0;
-    public get length(): number {
-      return this._length;
-    }
-    public set length(value: number) {
-      this._length = value;
-    }
+  _length: number = 0
+  public get length(): number {
+    return this._length
+  }
+  public set length(value: number) {
+    this._length = value
+  }
 
-    _capacity: number = 0
-    public get capacity(): number {
-      return this._capacity;
-    }
-    private set capacity(value: number) {
-      this._capacity = value;
-    }
-    public unsafeSetCapacity(value: number): void {
-      this.capacity = value
-    }
+  _capacity: number = 0
+  public get capacity(): number {
+    return this._capacity
+  }
+  private set capacity(value: number) {
+    this._capacity = value
+  }
+  public unsafeSetCapacity(value: number): void {
+    this.capacity = value
+  }
 }
 
 function g(_: number): void {}
@@ -48,4 +48,4 @@ function f() {
   g(d.length)
   g(d.capacity)
   g(D.length)
-}
+}

snapshots/input/syntax/src/type-alias.ts

@@ -1,20 +1,20 @@
 type S = string
 
-const s: S = ""
+const s: S = ''
 
 class C<T> {
-    t: T
+  t: T
 }
 type Cstring = C<string>
 
 const cs: Cstring = new C<string>()
 
 class D<T, U> {
-    t: T
-    u: U
+  t: T
+  u: U
 }
 type DT<T> = D<T, string> // partially specialized
 type DU<U> = D<string, DU<U>> // recursive!
 
 const dt: DT<string> = new D()
-const du: DU<string> = new D()
+const du: DU<string> = new D()

snapshots/output/pure-js/src/main.js

@@ -3,9 +3,9 @@
 //             ^ definition pure-js 1.0.0 src/`main.js`/fib().(n)
     if (n <= 1) {
 //      ^ reference pure-js 1.0.0 src/`main.js`/fib().(n)
-      return 0;
+      return 0
     }
-    return fib(n - 1) + fib(n - 2);
+    return fib(n - 1) + fib(n - 2)
 //         ^^^ reference pure-js 1.0.0 src/`main.js`/fib().
 //             ^ reference pure-js 1.0.0 src/`main.js`/fib().(n)
 //                      ^^^ reference pure-js 1.0.0 src/`main.js`/fib().
@@ -15,7 +15,7 @@
   function print_fib(a) {
 //         ^^^^^^^^^ definition pure-js 1.0.0 src/`main.js`/print_fib().
 //                   ^ definition pure-js 1.0.0 src/`main.js`/print_fib().(a)
-    console.log(fib(a));
+    console.log(fib(a))
 //  ^^^^^^^ reference typescript 4.6.2 lib/`lib.dom.d.ts`/console.
 //  ^^^^^^^ reference @types/node 17.0.14 `globals.d.ts`/console.
 //  ^^^^^^^ reference @types/node 17.0.14 `console.d.ts`/`'node:console'`/global/console/
@@ -26,68 +26,71 @@
 //                  ^ reference pure-js 1.0.0 src/`main.js`/print_fib().(a)
   }
 
-  var y = "Hello";
+  var y = 'Hello'
 //    ^ definition pure-js 1.0.0 src/`main.js`/y.
   function capture() {
 //         ^^^^^^^ definition pure-js 1.0.0 src/`main.js`/capture().
-    return y;
+    return y
 //         ^ reference pure-js 1.0.0 src/`main.js`/y.
   }
-  const capture_lambda = () => { return y; };
+  const capture_lambda = () => {
 //      ^^^^^^^^^^^^^^ definition pure-js 1.0.0 src/`main.js`/capture_lambda.
-//                                      ^ reference pure-js 1.0.0 src/`main.js`/y.
+    return y
+//         ^ reference pure-js 1.0.0 src/`main.js`/y.
+  }
 
-  for (var i = 0; i <= 10; i++) {
+  for (var i = 0; i <= 10; i++) {}
 //         ^ definition local 2
 //                ^ reference local 2
 //                         ^ reference local 2
-  }
 
   for (const x of [1, 2, 3]) {
 //           ^ definition local 5
   }
 
-  var a = 0;
+  var a = 0
 //    ^ definition pure-js 1.0.0 src/`main.js`/a.
-  var a = 1;
+  var a = 1
 //    ^ definition pure-js 1.0.0 src/`main.js`/a.
-  print_fib(a);
+  print_fib(a)
 //^^^^^^^^^ reference pure-js 1.0.0 src/`main.js`/print_fib().
 //          ^ reference pure-js 1.0.0 src/`main.js`/a.
 //          ^ reference pure-js 1.0.0 src/`main.js`/a.
 
-  function forever() { return forever(); }
+  function forever() {
 //         ^^^^^^^ definition pure-js 1.0.0 src/`main.js`/forever().
-//                            ^^^^^^^ reference pure-js 1.0.0 src/`main.js`/forever().
+    return forever()
+//         ^^^^^^^ reference pure-js 1.0.0 src/`main.js`/forever().
+  }
 
   function use_before_def() {
 //         ^^^^^^^^^^^^^^ definition pure-js 1.0.0 src/`main.js`/use_before_def().
-    print_fib(n);
+    print_fib(n)
 //  ^^^^^^^^^ reference pure-js 1.0.0 src/`main.js`/print_fib().
 //            ^ reference local 8
-    var n = 10;
+    var n = 10
 //      ^ definition local 8
 
     if (forever()) {
 //      ^^^^^^^ reference pure-js 1.0.0 src/`main.js`/forever().
-      var m = 10;
+      var m = 10
 //        ^ definition local 11
     }
-    print_fib(m);
+    print_fib(m)
 //  ^^^^^^^^^ reference pure-js 1.0.0 src/`main.js`/print_fib().
 //            ^ reference local 11
   }
 
   function var_function_scope() {
 //         ^^^^^^^^^^^^^^^^^^ definition pure-js 1.0.0 src/`main.js`/var_function_scope().
-    var k = 0;
+    var k = 0
 //      ^ definition local 14
     if (forever()) {
 //      ^^^^^^^ reference pure-js 1.0.0 src/`main.js`/forever().
-      var k = 1;
+      var k = 1
 //        ^ definition local 14
     }
-    print_fib(k);
+    print_fib(k)
 //  ^^^^^^^^^ reference pure-js 1.0.0 src/`main.js`/print_fib().
 //            ^ reference local 14
 //            ^ reference local 17