Merge pull request #744 from Wholesomebruh/skipped

src/handlers/comment/action.ts

@@ -40,8 +40,8 @@ export const handleComment = async (): Promise<void> => {
       const feature = config.command.find((e) => e.name === id.split("/")[1]);
 
       if (!feature?.enabled && id !== IssueCommentCommands.HELP) {
-        logger.info(`Skipping '${id}' because it is disabled on this repo`);
-        await callback(issue.number, `Skipping \`${id}\` because it is disabled on this repo`, payload.action, payload.comment);
+        logger.info(`Skipping '${id}' because it is disabled on this repo.`);
+        await callback(issue.number, `Skipping \`${id}\` because it is disabled on this repo.`, payload.action, payload.comment);
         continue;
       }
 

src/handlers/comment/handlers/assign.ts

@@ -152,7 +152,7 @@ const getMultiplierInfoToDisplay = async (senderLogin: string, org_id: string, i
   } else {
     _multiplierToDisplay = multiplier;
     _reasonToDisplay = reason;
-    _bountyToDisplay = `Permit generation skipped since price label is not set`;
+    _bountyToDisplay = `Permit generation disabled because price label is not set.`;
     const issueDetailed = bountyInfo(issue);
     if (issueDetailed.priceLabel) {
       _bountyToDisplay = (+issueDetailed.priceLabel.substring(7, issueDetailed.priceLabel.length - 4) * value).toString() + " USD";

src/handlers/comment/handlers/index.ts

@@ -90,7 +90,7 @@ export const issueCreatedCallback = async (): Promise<void> => {
   const { assistivePricing } = config.mode;
 
   if (!assistivePricing) {
-    logger.info("Skipping adding label to issue because assistive pricing is disabled");
+    logger.info("Skipping adding label to issue because assistive pricing is disabled.");
     return;
   }
 

src/handlers/payout/action.ts

@@ -39,7 +39,7 @@ export const handleIssueClosed = async () => {
   if (accessControl.organization) {
     const userHasPermission = await checkUserPermissionForRepoAndOrg(payload.sender.login, context);
 
-    if (!userHasPermission) return "Permit generation skipped because this issue has been closed by an external contributor.";
+    if (!userHasPermission) return "Permit generation disabled because this issue has been closed by an external contributor.";
   }
 
   const comments = await getAllIssueComments(issue.number);
@@ -94,19 +94,19 @@ export const handleIssueClosed = async () => {
     return;
   }
   if (privateKey == "") {
-    logger.info("Permit generation skipped because wallet private key is not set");
-    return "Permit generation skipped because wallet private key is not set";
+    logger.info("Permit generation disabled because wallet private key is not set.");
+    return "Permit generation disabled because wallet private key is not set.";
   }
   if (issue.state_reason !== StateReason.COMPLETED) {
-    logger.info("Permit generation skipped because this is marked as unplanned.");
-    return "Permit generation skipped because this is marked as unplanned.";
+    logger.info("Permit generation disabled because this is marked as unplanned.");
+    return "Permit generation disabled because this is marked as unplanned.";
   }
 
   logger.info(`Checking if the issue is a parent issue.`);
   if (issue.body && isParentIssue(issue.body)) {
-    logger.error("Permit generation skipped since the issue is identified as parent issue.");
+    logger.error("Permit generation disabled because this is a collection of issues.");
     await clearAllPriceLabelsOnIssue();
-    return "Permit generation skipped since the issue is identified as parent issue.";
+    return "Permit generation disabled because this is a collection of issues.";
   }
 
   logger.info(`Handling issues.closed event, issue: ${issue.number}`);
@@ -118,7 +118,7 @@ export const handleIssueClosed = async () => {
       if (res) {
         if (res[1] === "false") {
           logger.info(`Skipping to generate permit2 url, reason: autoPayMode for this issue: false`);
-          return `Permit generation skipped since automatic payment for this issue is disabled.`;
+          return `Permit generation disabled because automatic payment for this issue is disabled.`;
         }
         break;
       }
@@ -127,25 +127,25 @@ export const handleIssueClosed = async () => {
 
   if (paymentPermitMaxPrice == 0 || !paymentPermitMaxPrice) {
     logger.info(`Skipping to generate permit2 url, reason: { paymentPermitMaxPrice: ${paymentPermitMaxPrice}}`);
-    return `Permit generation skipped since paymentPermitMaxPrice is 0`;
+    return `Permit generation disabled because paymentPermitMaxPrice is 0.`;
   }
 
   const issueDetailed = bountyInfo(issue);
   if (!issueDetailed.isBounty) {
-    logger.info(`Skipping... its not a bounty`);
-    return `Permit generation skipped since this issue didn't qualify as bounty`;
+    logger.info(`Skipping... its not a bounty.`);
+    return `Permit generation disabled because this issue didn't qualify as bounty.`;
   }
 
   const assignees = issue?.assignees ?? [];
   const assignee = assignees.length > 0 ? assignees[0] : undefined;
   if (!assignee) {
-    logger.info("Skipping to proceed the payment because `assignee` is undefined");
-    return `Permit generation skipped since assignee is undefined`;
+    logger.info("Skipping to proceed the payment because `assignee` is undefined.");
+    return `Permit generation disabled because assignee is undefined.`;
   }
 
   if (!issueDetailed.priceLabel) {
     logger.info("Skipping to proceed the payment because price not set");
-    return `Permit generation skipped since price label is not set`;
+    return `Permit generation disabled because price label is not set.`;
   }
 
   const recipient = await getWalletAddress(assignee.login);
@@ -164,8 +164,8 @@ export const handleIssueClosed = async () => {
 
   let priceInEth = new Decimal(issueDetailed.priceLabel.substring(7, issueDetailed.priceLabel.length - 4)).mul(multiplier);
   if (priceInEth.gt(paymentPermitMaxPrice)) {
-    logger.info("Skipping to proceed the payment because bounty payout is higher than paymentPermitMaxPrice");
-    return `Permit generation skipped since issue's bounty is higher than ${paymentPermitMaxPrice}`;
+    logger.info("Skipping to proceed the payment because bounty payout is higher than paymentPermitMaxPrice.");
+    return `Permit generation disabled because issue's bounty is higher than ${paymentPermitMaxPrice}.`;
   }
 
   // if bounty hunter has any penalty then deduct it from the bounty
@@ -176,7 +176,7 @@ export const handleIssueClosed = async () => {
     const bountyAmountAfterPenalty = bountyAmount.sub(penaltyAmount);
     if (bountyAmountAfterPenalty.lte(0)) {
       await removePenalty(assignee.login, payload.repository.full_name, paymentToken, networkId.toString(), bountyAmount);
-      const msg = `Permit generation skipped because bounty amount after penalty is 0`;
+      const msg = `Permit generation disabled because bounty amount after penalty is 0.`;
       logger.info(msg);
       return msg;
     }
@@ -191,8 +191,8 @@ export const handleIssueClosed = async () => {
     `#### Task Assignee Reward\n### [ **[ CLAIM ${priceInEth} ${tokenSymbol.toUpperCase()} ]** ](${payoutUrl})\n` + "```" + shortenRecipient + "```";
   const permitComments = comments.filter((content) => content.body.includes("https://pay.ubq.fi?claim=") && content.user.type == UserType.Bot);
   if (permitComments.length > 0) {
-    logger.info(`Skip to generate a permit url because it has been already posted`);
-    return `Permit generation skipped because it was already posted to this issue.`;
+    logger.info(`Skip to generate a permit url because it has been already posted.`);
+    return `Permit generation disabled because it was already posted to this issue.`;
   }
   await deleteLabel(issueDetailed.priceLabel);
   await addLabelToIssue("Permitted");

src/handlers/payout/post.ts

@@ -31,7 +31,7 @@ export const incentivizeComments = async () => {
   }
 
   if (issue.state_reason !== StateReason.COMPLETED) {
-    logger.info("incentivizeComments: comment incentives skipped because the issue was not closed as completed");
+    logger.info("incentivizeComments: comment incentives disabled because the issue was not closed as completed.");
     return;
   }
 
@@ -143,7 +143,7 @@ export const incentivizeCreatorComment = async () => {
   }
 
   if (issue.state_reason !== StateReason.COMPLETED) {
-    logger.info("incentivizeCreatorComment: comment incentives skipped because the issue was not closed as completed");
+    logger.info("incentivizeCreatorComment: comment incentives disabled because the issue was not closed as completed.");
     return;
   }