Update dependency prettier to v3 (#390)

* Update dependency prettier to v3 * Upgrade prettier * Upgrade prettier * Update pre-commit --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Tim Jacomb <[email protected]>
timja · Feb 15, 2025 · 9db5240 · 9db5240
1 parent 7cf2da9
commit 9db5240
Show file tree

Hide file tree

Showing 19 changed files with 178 additions and 108 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -3,7 +3,7 @@ repos:
     rev: v1.2.0
     hooks:
       - id: helm-docs
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v2.7.1" # Use the sha or tag you want to point at
+  - repo: https://github.com/rbubley/mirrors-prettier
+    rev: "v3.5.1" # Use the sha or tag you want to point at
     hooks:
       - id: prettier
diff --git a/app/command-enabled.js b/app/command-enabled.js
@@ -32,7 +32,7 @@ export function labelEnabled(config, labels) {
     return {
       enabled: false,
       error: `${labels} doesn't match the allowed labels \`${labelConfig.allowedLabels.join(
-        ","
+        ",",
       )}\``,
     };
   }
@@ -70,7 +70,7 @@ export function removeLabelEnabled(config, labels) {
     return {
       enabled: false,
       error: `${labels} doesn't match the allowed labels \`${labelConfig.allowedLabels.join(
-        ","
+        ",",
       )}\``,
     };
   }

diff --git a/app/command-enabled.test.js b/app/command-enabled.test.js
@@ -44,7 +44,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label1"]
+        ["label1"],
       );
 
       expect(sut.enabled).toEqual(true);
@@ -59,7 +59,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label1"]
+        ["label1"],
       );
 
       expect(sut.enabled).toEqual(false);
@@ -75,7 +75,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label1"]
+        ["label1"],
       );
 
       expect(sut.enabled).toEqual(true);
@@ -90,7 +90,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label4"]
+        ["label4"],
       );
 
       expect(sut.enabled).toEqual(false);
@@ -105,7 +105,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label3", "label1", "label2"]
+        ["label3", "label1", "label2"],
       );
 
       expect(sut.enabled).toEqual(true);
@@ -120,7 +120,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label2", "label1", "label5"]
+        ["label2", "label1", "label5"],
       );
 
       expect(sut.enabled).toEqual(false);
@@ -187,7 +187,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label1"]
+        ["label1"],
       );
 
       expect(sut.enabled).toEqual(true);
@@ -202,7 +202,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label1"]
+        ["label1"],
       );
 
       expect(sut.enabled).toEqual(false);
@@ -218,7 +218,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label1"]
+        ["label1"],
       );
 
       expect(sut.enabled).toEqual(true);
@@ -233,7 +233,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label4"]
+        ["label4"],
       );
 
       expect(sut.enabled).toEqual(false);
@@ -248,7 +248,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label3", "label1", "label2"]
+        ["label3", "label1", "label2"],
       );
 
       expect(sut.enabled).toEqual(true);
@@ -263,7 +263,7 @@ describe("command-enabled", () => {
             },
           },
         },
-        ["label2", "label1", "label5"]
+        ["label2", "label1", "label5"],
       );
 
       expect(sut.enabled).toEqual(false);

diff --git a/app/commands/close-command.js b/app/commands/close-command.js
@@ -43,21 +43,21 @@ export class CloseCommand extends Command {
     }
 
     logger.info(
-      `Closing issue ${extractHtmlUrl(this.payload)}, reason: ${reason}`
+      `Closing issue ${extractHtmlUrl(this.payload)}, reason: ${reason}`,
     );
     try {
       await closeIssue(
         authToken,
         sourceRepo,
         extractLabelableId(this.payload),
-        reason
+        reason,
       );
     } catch (error) {
       logger.error(
         `Failed to close issue ${
           error.errors ? JSON.stringify(error.errors) : ""
         }`,
-        error
+        error,
       );
     }
   }

diff --git a/app/commands/label-command.js b/app/commands/label-command.js
@@ -37,22 +37,22 @@ export class LabelCommand extends Command {
     });
 
     logger.info(
-      `Labeling issue ${extractHtmlUrl(this.payload)} with labels ${labels}`
+      `Labeling issue ${extractHtmlUrl(this.payload)} with labels ${labels}`,
     );
     try {
       await addLabel(
         authToken,
         this.payload.repository.owner.login,
         sourceRepo,
         extractLabelableId(this.payload),
-        labels
+        labels,
       );
     } catch (error) {
       logger.error(
         `Failed to add label ${
           error.errors ? JSON.stringify(error.errors) : ""
         }`,
-        error
+        error,
       );
     }
   }

diff --git a/app/commands/remove-label-command.js b/app/commands/remove-label-command.js
@@ -37,23 +37,23 @@ export class RemoveLabelCommand extends Command {
 
     logger.info(
       `Removing label(s) from issue ${extractHtmlUrl(
-        this.payload
-      )}, labels ${labels}`
+        this.payload,
+      )}, labels ${labels}`,
     );
     try {
       await removeLabel(
         authToken,
         this.payload.repository.owner.login,
         sourceRepo,
         extractLabelableId(this.payload),
-        labels
+        labels,
       );
     } catch (error) {
       logger.error(
         `Failed to remove label ${
           error.errors ? JSON.stringify(error.errors) : ""
         }`,
-        error
+        error,
       );
     }
   }

diff --git a/app/commands/reopen-command.js b/app/commands/reopen-command.js
@@ -43,14 +43,14 @@ export class ReopenCommand extends Command {
       await reopenIssue(
         authToken,
         sourceRepo,
-        extractLabelableId(this.payload)
+        extractLabelableId(this.payload),
       );
     } catch (error) {
       logger.error(
         `Failed to reopen issue ${
           error.errors ? JSON.stringify(error.errors) : ""
         }`,
-        error
+        error,
       );
     }
   }

diff --git a/app/commands/reviewer-command.js b/app/commands/reviewer-command.js
@@ -36,12 +36,12 @@ export class ReviewerCommand extends Command {
 
     logger.info(
       `Requesting review for ${reviewerMatches} at ${extractHtmlUrl(
-        this.payload
-      )}`
+        this.payload,
+      )}`,
     );
     const reviewers = extractUsersAndTeams(
       this.payload.repository.owner.login,
-      reviewerMatches
+      reviewerMatches,
     );
     try {
       await requestReviewers(
@@ -50,14 +50,14 @@ export class ReviewerCommand extends Command {
         sourceRepo,
         extractLabelableId(this.payload),
         reviewers.users,
-        reviewers.teams
+        reviewers.teams,
       );
     } catch (error) {
       logger.error(
         `Failed to request reviewer ${
           error.errors ? JSON.stringify(error.errors) : ""
         }`,
-        error
+        error,
       );
     }
   }

diff --git a/app/commands/transfer-command.js b/app/commands/transfer-command.js
@@ -34,22 +34,22 @@ export class TransferCommand extends Command {
     });
 
     logger.info(
-      `Transferring issue ${extractHtmlUrl(this.payload)} to repo ${targetRepo}`
+      `Transferring issue ${extractHtmlUrl(this.payload)} to repo ${targetRepo}`,
     );
     try {
       await transferIssue(
         authToken,
         this.payload.repository.owner.login,
         sourceRepo,
         targetRepo,
-        extractLabelableId(this.payload)
+        extractLabelableId(this.payload),
       );
     } catch (error) {
       logger.error(
         `Failed to transfer issue ${
           error.errors ? JSON.stringify(error.errors) : ""
         }`,
-        error
+        error,
       );
     }
   }

diff --git a/app/comment-extractor.test.js b/app/comment-extractor.test.js
@@ -12,7 +12,7 @@ describe("extractors", () => {
           review: {
             body: "review body",
           },
-        })
+        }),
       ).toEqual("review body");
     });
 
@@ -22,7 +22,7 @@ describe("extractors", () => {
           comment: {
             body: "comment body",
           },
-        })
+        }),
       ).toEqual("comment body");
     });
   });
@@ -37,7 +37,7 @@ describe("extractors", () => {
           pull_request: {
             html_url: "https://github.com/some-org/some-repo/pull/1",
           },
-        })
+        }),
       ).toEqual("https://github.com/some-org/some-repo/pull/1");
     });
 
@@ -47,7 +47,7 @@ describe("extractors", () => {
           issue: {
             html_url: "https://github.com/some-org/some-repo/issue/1",
           },
-        })
+        }),
       ).toEqual("https://github.com/some-org/some-repo/issue/1");
     });
   });
@@ -62,7 +62,7 @@ describe("extractors", () => {
           pull_request: {
             node_id: "PR_aaaaaa",
           },
-        })
+        }),
       ).toEqual("PR_aaaaaa");
     });
 
@@ -72,7 +72,7 @@ describe("extractors", () => {
           issue: {
             node_id: "PR_abcdefgh",
           },
-        })
+        }),
       ).toEqual("PR_abcdefgh");
     });
   });

diff --git a/app/converters.test.js b/app/converters.test.js
@@ -13,7 +13,7 @@ describe("converters", () => {
     test("user and team", () => {
       const converted = extractUsersAndTeams(
         "test-org",
-        "@reviewer1,@test-org/team-1"
+        "@reviewer1,@test-org/team-1",
       );
 
       expect(converted).toEqual({
@@ -24,7 +24,7 @@ describe("converters", () => {
     test("multiple users and teams", () => {
       const converted = extractUsersAndTeams(
         "test-org",
-        "@reviewer1,@test-org/team-1,@reviewer2,@test-org/team-2"
+        "@reviewer1,@test-org/team-1,@reviewer2,@test-org/team-2",
       );
 
       expect(converted).toEqual({
@@ -35,7 +35,7 @@ describe("converters", () => {
     test("space separated users and teams", () => {
       const converted = extractUsersAndTeams(
         "test-org",
-        "@reviewer1 @test-org/team-1 @reviewer2 @test-org/team-2"
+        "@reviewer1 @test-org/team-1 @reviewer2 @test-org/team-2",
       );
 
       expect(converted).toEqual({
@@ -46,7 +46,7 @@ describe("converters", () => {
     test("space and comma separated users and teams", () => {
       const converted = extractUsersAndTeams(
         "test-org",
-        "@reviewer1 @test-org/team-1,@reviewer2 @test-org/team-2"
+        "@reviewer1 @test-org/team-1,@reviewer2 @test-org/team-2",
       );
 
       expect(converted).toEqual({
@@ -57,7 +57,7 @@ describe("converters", () => {
     test("remove empty entry in comma separated users and teams", () => {
       const converted = extractUsersAndTeams(
         "test-org",
-        "@reviewer1,@test-org/team-1,,@reviewer2,@test-org/team-2"
+        "@reviewer1,@test-org/team-1,,@reviewer2,@test-org/team-2",
       );
 
       expect(converted).toEqual({
@@ -68,7 +68,7 @@ describe("converters", () => {
     test("remove empty string entry in comma separated users and teams", () => {
       const converted = extractUsersAndTeams(
         "test-org",
-        "@reviewer1,@test-org/team-1, ,@reviewer2,@test-org/team-2"
+        "@reviewer1,@test-org/team-1, ,@reviewer2,@test-org/team-2",
       );
 
       expect(converted).toEqual({