Update prettier to v3 (#1229)

Author: ecraig12345

.prettierignore

@@ -15,6 +15,7 @@ docs/.vuepress/.temp/
 docs/.yarn
 /CHANGELOG.*
 change/
+coverage/
 dist/
 lib/
 LICENSE

change/change-337918d3-ea78-4c2e-9b54-f1fe3587642e.json

@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "type": "none",
+      "comment": "Update formatting",
+      "packageName": "beachball",
+      "email": "elcraig@microsoft.com",
+      "dependentChangeType": "none"
+    }
+  ]
+}

docs/concepts/groups.md

@@ -44,9 +44,9 @@ Example:
       "name": "group name",
       "include": ["packages/groupfoo/*"],
       "exclude": ["!packages/groupfoo/bar"],
-      "disallowedChangeTypes": ["major"]
-    }
-  ]
+      "disallowedChangeTypes": ["major"],
+    },
+  ],
 }
 ```
 

package.json

@@ -57,7 +57,7 @@
     "jest": "^29.0.0",
     "lage": "^2.15.0",
     "lint-staged": "^16.0.0",
-    "prettier": "~2.8.4",
+    "prettier": "~3.8.3",
     "syncpack": "^9.0.0",
     "ts-jest": "29.2.6",
     "typescript": "~6.0.0",

packages/beachball/src/__fixtures__/mockStdin.ts

@@ -17,7 +17,7 @@ export class MockStdin extends stream.Readable {
   };
   // This is a private property of the parent class. As of TS ES2022 output, it's necessary to use
   // `declare` to prevent TS from emitting an unset property which overwrites the one from the parent.
-  protected declare _readableState?: {
+  declare protected _readableState?: {
     length: number;
     ended: boolean;
     endEmitted: boolean;
@@ -162,7 +162,10 @@ class MockData {
   pos = 0;
   done = false;
 
-  constructor(private data: Buffer | string | null, public encoding?: BufferEncoding) {}
+  constructor(
+    private data: Buffer | string | null,
+    public encoding?: BufferEncoding
+  ) {}
 
   get length() {
     if (Buffer.isBuffer(this.data)) {

packages/beachball/src/__fixtures__/repositoryFactory.ts

@@ -187,9 +187,9 @@ export class RepositoryFactory {
           fixtureParam === 'single'
             ? getSinglePackageFixture()
             : fixtureParam === 'monorepo'
-            ? getMonorepoFixture()
-            : // Clone the user-provided fixture so it's safe to modify
-              cloneObject(fixtureParam),
+              ? getMonorepoFixture()
+              : // Clone the user-provided fixture so it's safe to modify
+                cloneObject(fixtureParam),
       };
     }
 

packages/beachball/src/__functional__/git/fetch.test.ts

@@ -25,7 +25,7 @@ describe('gitFetch', () => {
   const logs = initMockLogs();
 
   /** To speed things up, some tests only check the arguments and skip the git operation */
-  const noOpSuccess = () => ({ success: true, stdout: '', stderr: '', status: 0 } as GitProcessOutput);
+  const noOpSuccess = () => ({ success: true, stdout: '', stderr: '', status: 0 }) as GitProcessOutput;
 
   const { git: realGit, gitFailFast: _realGitFailFast } = jest.requireActual<typeof workspaceTools>('workspace-tools');
   const gitFailFast: typeof _realGitFailFast = (args, options) =>
@@ -81,7 +81,7 @@ describe('gitFetch', () => {
 
   it('returns error but does not throw or log on failure by default', () => {
     // This test uses controlled non-localized fake stdio so we can test the whole output
-    gitOverride = () => ({ success: false, stdout: 'some logs', stderr: 'oh no', status: 1 } as GitProcessOutput);
+    gitOverride = () => ({ success: false, stdout: 'some logs', stderr: 'oh no', status: 1 }) as GitProcessOutput;
 
     const res = gitFetch({ cwd: repo.rootPath, remote: '', branch: defaultBranchName });
     expect(res).toMatchObject({
@@ -125,7 +125,7 @@ describe('gitFetch', () => {
   });
 
   it('logs git output with failed fetch if verbose is true', () => {
-    gitOverride = () => ({ success: false, stdout: 'some logs', stderr: 'oh no', status: 1 } as GitProcessOutput);
+    gitOverride = () => ({ success: false, stdout: 'some logs', stderr: 'oh no', status: 1 }) as GitProcessOutput;
 
     const res = gitFetch({ cwd: repo.rootPath, verbose: true, remote: '', branch: defaultBranchName });
     expect(gitSpy).toHaveBeenCalledWith(['fetch'], expect.anything());

packages/beachball/src/__tests__/publish/bumpAndPush.test.ts

@@ -58,8 +58,8 @@ describe('bumpAndPush', () => {
   /** Create a mock workspace-tools git() result */
   function makeGitResult(opts: { success: boolean; output?: string }): GitProcessOutput {
     return {
-      stderr: opts.success ? '' : opts.output ?? '',
-      stdout: opts.success ? opts.output ?? '' : '',
+      stderr: opts.success ? '' : (opts.output ?? ''),
+      stdout: opts.success ? (opts.output ?? '') : '',
       success: opts.success,
       status: opts.success ? 0 : 1,
     } as GitProcessOutput;

packages/beachball/src/changefile/isPackageIncluded.ts

@@ -14,13 +14,13 @@ export function isPackageIncluded(
   const reason = !packageInfo
     ? 'no corresponding package found'
     : packageInfo.private
-    ? `${packageInfo.name} is private`
-    : // This is a package-only option (can't be set at repo level or via CLI)
-    packageInfo.packageOptions?.shouldPublish === false
-    ? `${packageInfo.name} has beachball.shouldPublish=false`
-    : !scopedPackages.has(packageInfo.name)
-    ? `${packageInfo.name} is out of scope`
-    : ''; // not ignored
+      ? `${packageInfo.name} is private`
+      : // This is a package-only option (can't be set at repo level or via CLI)
+        packageInfo.packageOptions?.shouldPublish === false
+        ? `${packageInfo.name} has beachball.shouldPublish=false`
+        : !scopedPackages.has(packageInfo.name)
+          ? `${packageInfo.name} is out of scope`
+          : ''; // not ignored
 
   return { isIncluded: !reason, reason };
 }

packages/beachball/src/changefile/readChangeFiles.ts

@@ -93,8 +93,8 @@ export function readChangeFiles(
       const warningType = !packageInfos[change.packageName]
         ? 'nonexistent'
         : packageInfos[change.packageName].private
-        ? 'private'
-        : undefined;
+          ? 'private'
+          : undefined;
       if (warningType) {
         const resolution = options.groupChanges ? 'remove the entry from this file' : 'delete this file';
         console.warn(