Add dangling commas to functions (new prettier default)

WebKit · Mar 7, 2024 · 572701f · 572701f
1 parent eff6a5a
commit 572701f
Show file tree

Hide file tree

Showing 23 changed files with 46 additions and 47 deletions.
diff --git a/.prettierrc b/.prettierrc
@@ -1,5 +1,4 @@
 {
     "printWidth": 250,
-    "tabWidth": 4,
-    "trailingComma": "es5"
+    "tabWidth": 4
 }
diff --git a/resources/main.mjs b/resources/main.mjs
@@ -39,7 +39,7 @@ class MainBenchmarkClient {
                 message,
                 params.suites,
                 "\nValid values:",
-                Suites.map((each) => each.name)
+                Suites.map((each) => each.name),
             );
 
             return false;

diff --git a/resources/metric-ui.mjs b/resources/metric-ui.mjs
@@ -32,7 +32,7 @@ export function renderMetricView(viewParams) {
                     <td>±</td>
                     <td>${metric.deltaString}</td>
                     <td>${metric.unit}</td>
-                </tr>`
+                </tr>`,
         )
         .join("");
     return `

diff --git a/resources/scripts/sanitize-language/index.js b/resources/scripts/sanitize-language/index.js
@@ -20,7 +20,7 @@ async function getFiles(dir) {
         dirents.map((dirent) => {
             const res = resolve(dir, dirent.name);
             return dirent.isDirectory() ? getFiles(res) : res;
-        })
+        }),
     );
     return Array.prototype.concat(...files);
 }

diff --git a/resources/todomvc/architecture-examples/angular/src/app/todo-footer/todo-footer.component.ts b/resources/todomvc/architecture-examples/angular/src/app/todo-footer/todo-footer.component.ts
@@ -10,7 +10,7 @@ import { TodosService } from "../todos.service";
 export class TodoFooterComponent {
     constructor(
         private todosService: TodosService,
-        private location: Location
+        private location: Location,
     ) {}
 
     get todos(): Todo[] {

diff --git a/resources/todomvc/architecture-examples/angular/src/app/todo-list/todo-list.component.ts b/resources/todomvc/architecture-examples/angular/src/app/todo-list/todo-list.component.ts
@@ -10,7 +10,7 @@ import { TodosService } from "../todos.service";
 export class TodoListComponent {
     constructor(
         private todosService: TodosService,
-        private location: Location
+        private location: Location,
     ) {}
 
     get todos(): Todo[] {

diff --git a/resources/todomvc/architecture-examples/backbone/src/views/app-view.js b/resources/todomvc/architecture-examples/backbone/src/views/app-view.js
@@ -64,7 +64,7 @@ var app = app || {};
                     this.statsTemplate({
                         completed: completed,
                         remaining: remaining,
-                    })
+                    }),
                 );
 
                 this.$(".filters li a")

diff --git a/resources/todomvc/architecture-examples/lit/src/lib/todo-list.ts b/resources/todomvc/architecture-examples/lit/src/lib/todo-list.ts
@@ -102,7 +102,7 @@ export class TodoList extends LitElement {
                 ${repeat(
         this.todoList?.filtered() ?? [],
         (todo) => todo.id,
-        (todo, index) => html`<todo-item data-priority=${4 - (index % 5)} .todoId=${todo.id} .text=${todo.text} .completed=${todo.completed}></todo-item>`
+        (todo, index) => html`<todo-item data-priority=${4 - (index % 5)} .todoId=${todo.id} .text=${todo.text} .completed=${todo.completed}></todo-item>`,
     )}
             </ul>
         `;

diff --git a/resources/todomvc/architecture-examples/react-redux/src/components/footer.test.jsx b/resources/todomvc/architecture-examples/react-redux/src/components/footer.test.jsx
@@ -10,7 +10,7 @@ const setup = (propOverrides) => {
             filter: SHOW_ALL,
             onClearCompleted: jest.fn(),
         },
-        propOverrides
+        propOverrides,
     );
 
     const { rerender } = render(<Footer {...props} />);

diff --git a/resources/todomvc/architecture-examples/react-redux/src/components/main.test.jsx b/resources/todomvc/architecture-examples/react-redux/src/components/main.test.jsx
@@ -36,13 +36,13 @@ const setup = (propOverrides) => {
             activeCount,
             visibleTodos,
         },
-        propOverrides
+        propOverrides,
     );
 
     const { rerender } = render(
         <HashRouter>
             <WrappedComponent {...props} />
-        </HashRouter>
+        </HashRouter>,
     );
 
     return {

diff --git a/resources/todomvc/architecture-examples/react-redux/src/components/text-input.test.jsx b/resources/todomvc/architecture-examples/react-redux/src/components/text-input.test.jsx
@@ -11,7 +11,7 @@ const setup = (propOverrides) => {
             editing: false,
             newTodo: false,
         },
-        propOverrides
+        propOverrides,
     );
 
     const { rerender } = render(<TextInput {...props} />);

diff --git a/resources/todomvc/architecture-examples/react-redux/src/index.js b/resources/todomvc/architecture-examples/react-redux/src/index.js
@@ -14,5 +14,5 @@ render(
             <Route path="*" component={App} />
         </HashRouter>
     </Provider>,
-    document.getElementById("root")
+    document.getElementById("root"),
 );
diff --git a/resources/todomvc/architecture-examples/react-redux/src/reducers/todos.test.js b/resources/todomvc/architecture-examples/react-redux/src/reducers/todos.test.js
@@ -11,7 +11,7 @@ describe("todos reducer", () => {
             todos([], {
                 type: types.ADD_TODO,
                 text: "Run the tests",
-            })
+            }),
         ).toMatchObject([
             {
                 completed: false,
@@ -31,8 +31,8 @@ describe("todos reducer", () => {
                 {
                     type: types.ADD_TODO,
                     text: "Run the tests",
-                }
-            )
+                },
+            ),
         ).toMatchObject([
             {
                 completed: false,
@@ -61,8 +61,8 @@ describe("todos reducer", () => {
                 {
                     type: types.ADD_TODO,
                     text: "Fix the tests",
-                }
-            )
+                },
+            ),
         ).toMatchObject([
             {
                 text: "Use Redux",
@@ -97,8 +97,8 @@ describe("todos reducer", () => {
                 {
                     type: types.DELETE_TODO,
                     id: 1,
-                }
-            )
+                },
+            ),
         ).toEqual([
             {
                 text: "Use Redux",
@@ -127,8 +127,8 @@ describe("todos reducer", () => {
                     type: types.EDIT_TODO,
                     text: "Fix the tests",
                     id: 1,
-                }
-            )
+                },
+            ),
         ).toEqual([
             {
                 text: "Fix the tests",
@@ -161,8 +161,8 @@ describe("todos reducer", () => {
                 {
                     type: types.TOGGLE_TODO,
                     id: 1,
-                }
-            )
+                },
+            ),
         ).toEqual([
             {
                 text: "Run the tests",
@@ -194,8 +194,8 @@ describe("todos reducer", () => {
                 ],
                 {
                     type: types.TOGGLE_ALL,
-                }
-            )
+                },
+            ),
         ).toMatchObject([
             {
                 text: "Run the tests",
@@ -226,8 +226,8 @@ describe("todos reducer", () => {
                 ],
                 {
                     type: types.TOGGLE_ALL,
-                }
-            )
+                },
+            ),
         ).toEqual([
             {
                 text: "Run the tests",
@@ -259,8 +259,8 @@ describe("todos reducer", () => {
                 ],
                 {
                     type: types.CLEAR_COMPLETED,
-                }
-            )
+                },
+            ),
         ).toEqual([
             {
                 text: "Use Redux",
@@ -295,7 +295,7 @@ describe("todos reducer", () => {
                     completed: false,
                     text: "Write tests",
                 },
-            ])
+            ]),
         ).toMatchObject([
             {
                 text: "Write tests",

diff --git a/resources/todomvc/architecture-examples/react/src/index.js b/resources/todomvc/architecture-examples/react/src/index.js
@@ -11,5 +11,5 @@ render(
             <Route path="*" element={<App />} />
         </Routes>
     </HashRouter>,
-    document.getElementById("root")
+    document.getElementById("root"),
 );
diff --git a/resources/todomvc/architecture-examples/react/src/todo/components/input.jsx b/resources/todomvc/architecture-examples/react/src/todo/components/input.jsx
@@ -35,7 +35,7 @@ export function Input({ onSubmit, placeholder, label, defaultValue, onBlur }) {
                 e.target.value = "";
             }
         },
-        [onSubmit]
+        [onSubmit],
     );
 
     return (

diff --git a/resources/todomvc/architecture-examples/react/src/todo/components/item.jsx b/resources/todomvc/architecture-examples/react/src/todo/components/item.jsx
@@ -28,7 +28,7 @@ export const Item = memo(function Item({ todo, dispatch, index }) {
 
             setIsWritable(false);
         },
-        [id, removeItem, updateItem]
+        [id, removeItem, updateItem],
     );
 
     return (

diff --git a/resources/todomvc/architecture-examples/react/src/todo/components/main.jsx b/resources/todomvc/architecture-examples/react/src/todo/components/main.jsx
@@ -18,7 +18,7 @@ export function Main({ todos, dispatch }) {
 
                 return todo;
             }),
-        [todos, route]
+        [todos, route],
     );
 
     const toggleAll = useCallback((e) => dispatch({ type: "TOGGLE_ALL", payload: { completed: e.target.checked } }), [dispatch]);

diff --git a/resources/todomvc/big-dom-generator/src/components/popovers/backlog.jsx b/resources/todomvc/big-dom-generator/src/components/popovers/backlog.jsx
@@ -27,7 +27,7 @@ export const BacklogPopOver = ({ className }) => {
                         </button>
                     </ActionGroup>
                 </div>
-            </li>
+            </li>,
         );
     }
 

diff --git a/resources/todomvc/big-dom-generator/src/components/popovers/popover.jsx b/resources/todomvc/big-dom-generator/src/components/popovers/popover.jsx
@@ -29,7 +29,7 @@ export const OptionsPopOver = ({ numOptions, className, startRight }) => {
         options.push(
             <li key={i} className="spectrum-Menu-item" role="menuitem" tabIndex="0">
                 <span className="spectrum-Menu-itemLabel">Hidden Option {i}</span>
-            </li>
+            </li>,
         );
     }
     const classNamePopOver = classnames("spectrum-Popover", { "spectrum-Popover--bottom": !startRight }, { "spectrum-Popover--bottom-right": startRight }, className);

diff --git a/resources/todomvc/vanilla-examples/javascript-es5/src/store.js b/resources/todomvc/vanilla-examples/javascript-es5/src/store.js
@@ -57,7 +57,7 @@
                 }
 
                 return true;
-            })
+            }),
         );
     };
 

diff --git a/resources/todomvc/vanilla-examples/javascript-es6-webpack/src/store.js b/resources/todomvc/vanilla-examples/javascript-es6-webpack/src/store.js
@@ -56,7 +56,7 @@ export class Store {
                 }
 
                 return true;
-            })
+            }),
         );
     }
 

diff --git a/...anilla-examples/javascript-web-components/src/components/todo-item/todo-item.component.js b/...anilla-examples/javascript-web-components/src/components/todo-item/todo-item.component.js
@@ -94,7 +94,7 @@ class TodoItem extends HTMLElement {
             new CustomEvent("toggle-item", {
                 detail: { id: this.itemid, completed: this.toggleInput.checked },
                 bubbles: true,
-            })
+            }),
         );
     }
 
@@ -105,7 +105,7 @@ class TodoItem extends HTMLElement {
             new CustomEvent("remove-item", {
                 detail: { id: this.itemid },
                 bubbles: true,
-            })
+            }),
         );
         this.remove();
     }
@@ -120,7 +120,7 @@ class TodoItem extends HTMLElement {
                     new CustomEvent("update-item", {
                         detail: { id: this.itemid, title: event.target.value },
                         bubbles: true,
-                    })
+                    }),
                 );
             }
         }
@@ -173,7 +173,7 @@ class TodoItem extends HTMLElement {
                 callbacks: {
                     [" "]: this.startEdit, // this feels weird
                 },
-            })
+            }),
         );
 
         this.addListeners();

diff --git a/...la-examples/javascript-web-components/src/components/todo-topbar/todo-topbar.component.js b/...la-examples/javascript-web-components/src/components/todo-topbar/todo-topbar.component.js
@@ -36,7 +36,7 @@ class TodoTopbar extends HTMLElement {
         this.dispatchEvent(
             new CustomEvent("toggle-all", {
                 detail: { completed: event.target.checked },
-            })
+            }),
         );
     }
 
@@ -51,7 +51,7 @@ class TodoTopbar extends HTMLElement {
                     title: event.target.value,
                     completed: false,
                 },
-            })
+            }),
         );
 
         event.target.value = "";
@@ -112,7 +112,7 @@ class TodoTopbar extends HTMLElement {
                 callbacks: {
                     ["Enter"]: this.addItem,
                 },
-            })
+            }),
         );
 
         this.updateDisplay();
-Original file line number
+Diff line change
@@ Expand Up / @@ -27,7 +27,7 @@ export const BacklogPopOver = ({ className }) => { @@
                             </button>
                         </ActionGroup>
                     </div>
-                </li>
+                </li>,
             );
         }
@@ Expand Down @@