Skip labeling when no tickernumber was found

index.js

@@ -3,34 +3,29 @@ import { getOctokit, context } from "@actions/github";
 import computePrTitle from "./lib/compute-proper-title";
 import findIssueInBranch from "./lib/find-issue";
 
+// Find token from request
+const token = getInput("token", { required: true });
+const octokit = getOctokit(token);
+
+// Prepare reusable query
+const prQuery = {
+    owner: context.repo.owner,
+    repo: context.repo.repo,
+    pull_number: context.issue.number,
+};
+
+let ticketNumber = null;
+
 async function updatePrTitle() {
-    startGroup("Start processing PR...");
+    startGroup("Start updating PR title...");
 
     try {
-        // Find token from request
-        const token = getInput("token", { required: true });
-        const octokit = getOctokit(token);
-
         // Find input ticket, if any
         const inputTicket = getInput("ticket-number", { required: false });
 
-        // Prepare reusable query
-        const prQuery = {
-            owner: context.repo.owner,
-            repo: context.repo.repo,
-            pull_number: context.issue.number,
-        };
-
         // Fetch PR
         const pullRequest = await octokit.rest.pulls.get(prQuery);
 
-        // Determine the ticket number from the input, the branch or the title
-        const ticketNumber = inputTicket ||
-            findIssueInBranch(pullRequest.data.head.ref) ||
-            findIssueInBranch(pullRequest.data.title);
-
-        info(`Ticket number "${ticketNumber}" found.`);
-
         // Get the current title as shorthand for comparison
         const prHeadRef = pullRequest.data.head.ref;
         const currentTitle = String(pullRequest.data.title);
@@ -40,6 +35,13 @@ async function updatePrTitle() {
             return;
         }
 
+        // Determine the ticket number from the input, the branch or the title
+        ticketNumber = inputTicket ||
+            findIssueInBranch(pullRequest.data.head.ref) ||
+            findIssueInBranch(pullRequest.data.title);
+
+        info(`Ticket number: "${ticketNumber}"`);
+
         // Let the system compute the right title
         const properTitle = computePrTitle(prHeadRef, currentTitle, ticketNumber);
 
@@ -63,16 +65,30 @@ async function updatePrTitle() {
             ...prQuery,
             title: properTitle,
         });
+    } catch (error) {
+        setFailed(error.message);
+    }
+
+    endGroup();
+}
 
+async function updatePrLabels() {
+    startGroup("Start updating Labels on PR...");
         const assignDefaultLabel = getInput("label", { required: false });
 
-        if (assignDefaultLabel == 'false') {
+        if (assignDefaultLabel == "false") {
             info(`Skip labeling PR`);
 
             return;
         }
 
-        // Assign label to PR
+        if (!ticketNumber) {
+            info(`No ticket number found, don't label PR`);
+
+            return;
+        }
+
+        // Fetch labels
         const labels = await octokit.rest.issues.listLabelsForRepo({
             owner: prQuery.owner,
             repo: prQuery.repo,
@@ -96,11 +112,9 @@ async function updatePrTitle() {
                 labels: [matchingLabelWithTicket.name],
             });
         }
-    } catch (error) {
-        setFailed(error.message);
-    }
 
     endGroup();
 }
 
 updatePrTitle();
+updatePrLabels();