Merge pull request #2893 from Azure/dpwatrous/prettier-upgrade

Update prettier to 3.x

Author: dpwatrous

.prettierrc.json

@@ -1,3 +1,12 @@
 {
-  "endOfLine": "auto"
+  "endOfLine": "auto",
+  "trailingComma": "es5",
+  "overrides": [
+    {
+      "files": ["*.jsonc", "tsconfig.json"],
+      "options": {
+        "trailingComma": "none"
+      }
+    }
+  ]
 }

desktop/package-lock.json

@@ -45,7 +45,7 @@
     "markdownlint": "^0.25.1",
     "markdownlint-cli2": "^0.4.0",
     "npm-run-all": "^4.1.5",
-    "prettier": "^2.5.1",
+    "prettier": "^3.0.0",
     "typescript": "^4.9.0",
     "which": "^2.0.2"
   },

package-lock.json

@@ -18,7 +18,7 @@ import { Notifier } from "../notification";
  */
 export abstract class AbstractEnvironment<
     C extends EnvironmentConfig = EnvironmentConfig,
-    D extends DependencyFactories = DependencyFactories
+    D extends DependencyFactories = DependencyFactories,
 > implements Environment<C>
 {
     abstract readonly name: EnvironmentName;

package.json

@@ -15,7 +15,7 @@ let _currentEnvironment: Environment<EnvironmentConfig> | null = null;
  * @returns The current environment
  */
 export function getEnvironment<
-    C extends EnvironmentConfig = EnvironmentConfig
+    C extends EnvironmentConfig = EnvironmentConfig,
 >(): Environment<C> {
     if (!_currentEnvironment) {
         throw new Error(
@@ -60,7 +60,7 @@ export function inject<T>(id: string): T {
  * environment is not a mock.
  */
 export function getMockEnvironment<
-    C extends EnvironmentConfig = EnvironmentConfig
+    C extends EnvironmentConfig = EnvironmentConfig,
 >(): MockEnvironment<C> {
     if (!_currentEnvironment) {
         throw new Error(
@@ -103,7 +103,7 @@ export function initEnvironment<T extends Environment<EnvironmentConfig>>(
  */
 export function initMockEnvironment<
     C extends EnvironmentConfig = EnvironmentConfig,
-    D extends DependencyFactories = DependencyFactories
+    D extends DependencyFactories = DependencyFactories,
 >(
     configOverrides: Partial<C> = {},
     depFactoryOverrides: Partial<D> = {}

packages/bonito-core/src/environment/abstract-environment.ts

@@ -34,7 +34,7 @@ export const mockDependencyFactories: DependencyFactories = {
 };
 
 export class MockEnvironment<
-    C extends EnvironmentConfig
+    C extends EnvironmentConfig,
 > extends AbstractEnvironment<C, DependencyFactories> {
     name = EnvironmentName.Mock;
 

packages/bonito-core/src/environment/environment-util.ts

@@ -800,7 +800,7 @@ const allowedStates = ["NY", "New York", "CA", "California"];
 class ParkParameter<
     V extends FormValues,
     K extends ParameterName<V>,
-    D extends ParameterDependencies<V, "state">
+    D extends ParameterDependencies<V, "state">,
 > extends AbstractParameter<V, K, D> {
     async validateAsync(): Promise<ValidationStatus> {
         let status = await super.validateAsync();

packages/bonito-core/src/environment/mock-environment.ts

@@ -8,7 +8,7 @@ import { ValidationStatus } from "./validation-status";
  */
 export class BooleanParameter<
     V extends FormValues,
-    K extends ParameterName<V>
+    K extends ParameterName<V>,
 > extends AbstractParameter<V, K> {
     validateSync(): ValidationStatus {
         let status = super.validateSync();

packages/bonito-core/src/form/__tests__/form.spec.ts

@@ -61,7 +61,7 @@ export interface EntryInit<V extends FormValues> {
 
 export interface ValuedEntryInit<
     V extends FormValues,
-    K extends ParameterName<V>
+    K extends ParameterName<V>,
 > extends EntryInit<V> {
     value?: V[K];
 }