Merge pull request #705 from preactjs/prettier-update

chore: update prettier

Author: marvinhagemeister

.github/ISSUE_TEMPLATE/bug-report.md

@@ -1,14 +1,15 @@
 ---
 name: Bug report
 about: Create a report to help us improve
-title: ''
-labels: ''
-assignees: ''
+title: ""
+labels: ""
+assignees: ""
 ---
 
 - [ ] Check if updating to the latest version resolves the issue
 
 **Environment**
+
 - [ ] I am using `@preact/signals-core`
 - [ ] I am using `@preact/signals`
 - [ ] I am using `@preact/signals-react`

.github/ISSUE_TEMPLATE/feature-request.md

@@ -1,9 +1,9 @@
 ---
 name: Feature request
 about: Suggest an idea for this project
-title: ''
+title: ""
 labels: feature request
-assignees: ''
+assignees: ""
 ---
 
 **Describe the feature you'd love to see**

docs/vite.config.ts

@@ -25,7 +25,7 @@ export default defineConfig(env => ({
 		process.env.DEBUG
 			? preact({
 					exclude: /\breact/,
-			  })
+				})
 			: null,
 		multiSpa(["index.html", "demos/**/*.html"]),
 		unsetPreactAliases(),
@@ -68,7 +68,7 @@ export default defineConfig(env => ({
 						// one, as expected. I'm working around this by just mainly aliasing
 						// the package that needs to be resolved.
 						"@preact/signals-react/runtime": join(root, "react/runtime"),
-				  }
+					}
 				: packages(false),
 	},
 }));

karma.conf.js

@@ -318,7 +318,7 @@ module.exports = function (config) {
 							watched: false,
 							type: "js",
 						},
-				  ]
+					]
 				: []),
 			{
 				pattern:

package.json

@@ -86,7 +86,7 @@
     "microbundle": "^0.15.1",
     "mocha": "^10.0.0",
     "oxlint": "^1.3.0",
-    "prettier": "^2.7.1",
+    "prettier": "^3.6.2",
     "shx": "^0.3.4",
     "sinon": "^14.0.0",
     "sinon-chai": "^3.7.0",
@@ -98,7 +98,8 @@
     ]
   },
   "prettier": {
-    "arrowParens": "avoid"
+    "arrowParens": "avoid",
+    "trailingComma": "es5"
   },
   "pnpm": {
     "patchedDependencies": {

packages/react-transform/src/index.ts

@@ -433,10 +433,10 @@ function transformFunction(
 	const hookUsage = options.experimental?.noTryFinally
 		? UNMANAGED
 		: isHook
-		? MANAGED_HOOK
-		: isComponent
-		? MANAGED_COMPONENT
-		: UNMANAGED;
+			? MANAGED_HOOK
+			: isComponent
+				? MANAGED_COMPONENT
+				: UNMANAGED;
 
 	let newBody: BabelTypes.BlockStatement;
 	if (hookUsage !== UNMANAGED) {

packages/react/test/browser/react-router.test.tsx

@@ -17,9 +17,9 @@ const Routes = ReactRouter.Routes
 const Route = ReactRouter.Routes
 	? ReactRouter.Route
 	: // react-router-dom v5 requires the element prop to be passed as children.
-	  ({ element, ...props }: any) => (
+		({ element, ...props }: any) => (
 			<ReactRouter.Route {...props}>{element}</ReactRouter.Route>
-	  );
+		);
 
 describe("@preact/signals-react", () => {
 	let scratch: HTMLDivElement;