feat: added variables to logs #9
Conversation
claudiacodacy
commented
Dec 5, 2025
claudiacodacy
commented
- adding variables to the log for better degguing
github-actions
bot
added
the
codacy-review
Codacy's Analysis Summary0 new issue (≤ 0 issue) Review Pull Request in Codacy →
|
![codacy-production[bot]](https://avatars.githubusercontent.com/in/56611?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
Pull Request Overview
The PR passes all Codacy quality gates, with no new issues and a decrease in complexity. The changes focus on improving log messages for better debugging.
About this PR
- There's a small typo in the PR description. 'degguing' should be 'debugging'.
💡 Codacy uses AI. Check for mistakes.
| if (excludeKeywords.length > 0) { | ||
| if (excludeKeywords.some(k => matchesKeyword(title, k))) { | ||
| return log("Skipping PR with excluded keywords"); | ||
| return log("Skipping PR with excluded keywords"+`\ntitle = ${title}`+`\nexcludeKeywords = ${excludeKeywords}`); |
There was a problem hiding this comment.
⚪ LOW RISK
Suggestion: Combining string literals with template literals can be less readable. Consider using a single template literal for the entire log message.
This might be a simple fix:
| return log("Skipping PR with excluded keywords"+`\ntitle = ${title}`+`\nexcludeKeywords = ${excludeKeywords}`); | |
| return log(`Skipping PR with excluded keywords\ntitle = ${title}\nexcludeKeywords = ${excludeKeywords}`); |
| if (requiredKeywords.length > 0) { | ||
| if (!requiredKeywords.some(k => matchesKeyword(title, k))) { | ||
| return log("Skipping PR without required keywords"); | ||
| return log("Skipping PR without required keywords"+`\ntitle = ${title}`+`\nrequiredKeywords = ${requiredKeywords}`); |
There was a problem hiding this comment.
⚪ LOW RISK
Suggestion: Combining string literals with template literals can be less readable. Consider using a single template literal for the entire log message.
This might be a simple fix:
| return log("Skipping PR without required keywords"+`\ntitle = ${title}`+`\nrequiredKeywords = ${requiredKeywords}`); | |
| return log(`Skipping PR without required keywords\ntitle = ${title}\nrequiredKeywords = ${requiredKeywords}`); |
| const author = pr.user?.login || ""; | ||
| const isBot = pr.user?.type === "Bot" || author.endsWith("[bot]"); | ||
| if (isBot) return log("Skipping bot PR"); | ||
| if (isBot) return log("Skipping bot PR"+`\nauthor = ${author}`+`\nisBot = ${isBot}`); |
There was a problem hiding this comment.
⚪ LOW RISK
Suggestion: Combining string literals with template literals can be less readable. Consider using a single template literal for the entire log message.
| if (!includeDrafts) { | ||
| const isDraft = pr.draft === true; | ||
| if (isDraft) return log("Skipping draft PR"); | ||
| if (isDraft) return log("Skipping draft PR"+`\nisDraft = ${isDraft}`); |
There was a problem hiding this comment.
⚪ LOW RISK
Suggestion: Combining string literals with template literals can be less readable. Consider using a single template literal for the entire log message.
