Add prettier always arrowParens (#47)

roginfarrer · Apr 17, 2020 · c428108 · c428108
1 parent dea827f
commit c428108
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 9 deletions.
diff --git a/README.md b/README.md
@@ -67,7 +67,7 @@ function Demo() {
     <div>
       <button
         {...getToggleProps({
-          onClick: () => setOpen(oldOpen => !oldOpen),
+          onClick: () => setOpen((oldOpen) => !oldOpen),
         })}
       >
         {isExpanded ? 'Collapse' : 'Expand'}

diff --git a/package.json b/package.json
@@ -30,10 +30,12 @@
     }
   },
   "prettier": {
+    "bracketSpacing": true,
     "printWidth": 80,
     "semi": true,
     "singleQuote": true,
-    "trailingComma": "es5"
+    "trailingComma": "es5",
+    "arrowParens": "always"
   },
   "devDependencies": {
     "@babel/core": "^7.8.4",

diff --git a/src/hooks.ts b/src/hooks.ts
@@ -17,7 +17,7 @@ export function useControlledState({
     : stateExpanded;
   const toggleExpanded = useCallback(() => {
     if (!initiallyControlled.current) {
-      setStateExpanded(oldExpanded => !oldExpanded);
+      setStateExpanded((oldExpanded) => !oldExpanded);
     }
   }, []);
 
@@ -70,7 +70,7 @@ export function usePaddingWarning(element: RefObject<HTMLElement>): void {
   let warn = (el?: RefObject<HTMLElement>): void => {};
 
   if (__DEV__) {
-    warn = el => {
+    warn = (el) => {
       if (!el?.current) {
         return;
       }

diff --git a/src/index.ts b/src/index.ts
@@ -46,7 +46,7 @@ export default function useCollapse({
   );
   const [mountChildren, setMountChildren] = useState<boolean>(isExpanded);
   const mergeStyles = (newStyles: {}): void => {
-    setStyles(oldStyles => ({ ...oldStyles, ...newStyles }));
+    setStyles((oldStyles) => ({ ...oldStyles, ...newStyles }));
   };
 
   function getTransitionStyles(

diff --git a/src/utils.ts b/src/utils.ts
@@ -24,7 +24,7 @@ export function getElementHeight(
 
 // Helper function for render props. Sets a function to be called, plus any additional functions passed in
 export const callAll = (...fns: AnyFunction[]) => (...args: any[]): void =>
-  fns.forEach(fn => fn && fn(...args));
+  fns.forEach((fn) => fn && fn(...args));
 
 // https://github.com/mui-org/material-ui/blob/da362266f7c137bf671d7e8c44c84ad5cfc0e9e2/packages/material-ui/src/styles/transitions.js#L89-L98
 export function getAutoHeightDuration(height: number | string): number {
@@ -64,11 +64,11 @@ export function assignRef<RefValueType = any>(
 export function mergeRefs<RefValueType = any>(
   ...refs: (AssignableRef<RefValueType> | null | undefined)[]
 ) {
-  if (refs.every(ref => ref == null)) {
+  if (refs.every((ref) => ref == null)) {
     return null;
   }
   return (node: any) => {
-    refs.forEach(ref => {
+    refs.forEach((ref) => {
       assignRef(ref, node);
     });
   };

diff --git a/stories/basic.stories.tsx b/stories/basic.stories.tsx
@@ -24,7 +24,7 @@ export const Controlled = () => {
 
   return (
     <div>
-      <Toggle {...getToggleProps({ onClick: () => setOpen(old => !old) })}>
+      <Toggle {...getToggleProps({ onClick: () => setOpen((old) => !old) })}>
         {isExpanded ? 'Close' : 'Open'}
       </Toggle>
       <Collapse {...getCollapseProps()}>{excerpt}</Collapse>