EC-00: FIX: The Logical Error (since parameter) #1

Closed
Puranjay Savar Mattas wants to merge 2 commits from EC-00-Action-Bugs into main

🎟️ Evercatch Ticket

Resolves: EC-00

🛠️ Key Changes

  • Date Comparison: Changed from using the API since filter to a bash comparison using date +%s.
  • Safety Check: Added if [ -z "$issue" ]; then continue; fi inside the loop to prevent the crash if jq outputs an empty line.
  • Logic: The script now checks if the issue is actually older than 30 days before applying the label.

Author's Definition of Done

  • I have performed a self-review of my own code.
  • My code follows the style guidelines and architectural patterns of this project.
  • I have added or updated tests that prove my feature works or my fix is effective.
  • All new and existing tests pass locally with my changes.
  • I have made corresponding changes to the documentation (if applicable).

🔬 Reviewer's Guide & Additional Info (Click to Expand)

Where should the reviewer start?

🏷️ Type of Change

  • FEAT: A new feature
  • FIX: A bug fix
  • DOCS: Documentation only changes
  • STYLE: Code style changes (formatting, etc.)
  • REFACTOR: A code change that neither fixes a bug nor adds a feature
  • PERF: A code change that improves performance
  • TEST: Adding missing tests or correcting existing tests
  • CHORE: Changes to the build process or auxiliary tools


Evercatch Banner

### 🎟️ Evercatch Ticket <!-- Required. Link to the Jira ticket. Use 'Resolves' to auto-close. --> Resolves: [EC-00](<PROVIDE-LINK>) ### 🛠️ Key Changes - Date Comparison: Changed from using the API since filter to a bash comparison using date +%s. - Safety Check: Added if [ -z "$issue" ]; then continue; fi inside the loop to prevent the crash if jq outputs an empty line. - Logic: The script now checks if the issue is actually older than 30 days before applying the label. --- ### ✅ Author's Definition of Done <!-- Please check the boxes that apply. This is your declaration that the work is ready for review. --> - [x] I have performed a self-review of my own code. - [x] My code follows the style guidelines and architectural patterns of this project. - [x] I have added or updated tests that prove my feature works or my fix is effective. - [x] All new and existing tests pass locally with my changes. - [x] I have made corresponding changes to the documentation (if applicable). <br> <details> <summary><strong>🔬 Reviewer's Guide & Additional Info (Click to Expand)</strong></summary> --- #### Where should the reviewer start? <!-- Optional: Give the reviewer a hint! e.g., "Start with `src/services/NewService.ts` as that contains the core logic." --> #### 🏷️ Type of Change <!-- This provides context for the change. Please check one. --> - [ ] **FEAT**: A new feature - [x] **FIX**: A bug fix - [ ] **DOCS**: Documentation only changes - [ ] **STYLE**: Code style changes (formatting, etc.) - [ ] **REFACTOR**: A code change that neither fixes a bug nor adds a feature - [ ] **PERF**: A code change that improves performance - [ ] **TEST**: Adding missing tests or correcting existing tests - [ ] **CHORE**: Changes to the build process or auxiliary tools --- </details> --- ![Evercatch Banner](https://placehold.co/1200x300/080c12/00e5ff?text=Evercatch&font=montserrat)
Puranjay Savar Mattas added the type: fix label 2026-02-16 09:51:31 +00:00
Puranjay Savar Mattas self-assigned this 2026-02-16 09:51:31 +00:00
Puranjay Savar Mattas added 1 commit 2026-02-16 09:51:31 +00:00
FIX: The Logical Error (since parameter)
Some checks failed
PR Management Bot / pr-bot (pull_request) Failing after 2s
PR Title Checker / Validate PR Title Format (pull_request) Successful in 1s
08bb82b7ae
- Date Comparison: Changed from using the API since filter to a bash comparison using date +%s.
 - Safety Check: Added if [ -z "$issue" ]; then continue; fi inside the loop to prevent the crash if jq outputs an empty line.
 - Logic: The script now checks if the issue is actually older than 30 days before applying the label.
Author
Owner

Hi @psmattas, thanks for the contribution!

The title of this pull request does not follow the required format. This PR is currently blocked from merging.

To fix this, please update the title to match the following structure:
EC-[ISSUE_NUMBER]: [TYPE]: [Your Description]

  • Valid [TYPE] values are: FEAT, FIX, DOCS, REFACTOR, STYLE, PERF, TEST, CHORE
  • Example: EC-42: FEAT: Add new user login endpoint

Once you have updated the title, this check will re-run automatically and remove the blocking label.

Hi @psmattas, thanks for the contribution! The title of this pull request does not follow the required format. This PR is currently **blocked from merging**. To fix this, please update the title to match the following structure: **`EC-[ISSUE_NUMBER]: [TYPE]: [Your Description]`** - **Valid [TYPE] values are:** `FEAT`, `FIX`, `DOCS`, `REFACTOR`, `STYLE`, `PERF`, `TEST`, `CHORE` - **Example:** `EC-42: FEAT: Add new user login endpoint` Once you have updated the title, this check will re-run automatically and remove the blocking label. <!-- PR_TITLE_VALIDATOR_COMMENT -->
Puranjay Savar Mattas requested review from Owners 2026-02-16 09:51:56 +00:00
Puranjay Savar Mattas changed title from FIX: The Logical Error (since parameter) to EC-00: FIX: The Logical Error (since parameter) 2026-02-16 09:52:17 +00:00
Puranjay Savar Mattas removed review request for Owners 2026-02-16 09:59:14 +00:00
Puranjay Savar Mattas requested review from Evercatch Bot 2026-02-16 09:59:18 +00:00
Evercatch Bot approved these changes 2026-02-16 09:59:42 +00:00
Dismissed
Evercatch Bot left a comment
Member

Auto-Approved for PSMattas

Auto-Approved for PSMattas
Puranjay Savar Mattas added 1 commit 2026-02-16 11:18:23 +00:00
FIX: INTERNAL_ERROR in checkout
All checks were successful
PR Management Bot / pr-bot (pull_request) Successful in 14s
PR Title Checker / Validate PR Title Format (pull_request) Successful in 2s
4aca6997ff
- Grep Exit Code: In the Auto-label step, if grep finds no matching commit types, it exits with code 1. Since workflows run with set -e, this will cause the job to fail immediately if no labels are found.
- Self-Review Request: In the Auto-assign reviewers step, you are hardcoding psmattas as a reviewer. Since psmattas is also the author (based on the logs), the API will return a 422 Unprocessable Entity error because an author cannot review their own PR.
Puranjay Savar Mattas dismissed evercatch-bot's review 2026-02-16 11:18:24 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

Evercatch Bot requested review from Evercatch Bot 2026-02-16 11:21:26 +00:00
Evercatch Bot approved these changes 2026-02-16 11:21:31 +00:00
Dismissed
Evercatch Bot left a comment
Member

Auto-Approved for PSMattas

Auto-Approved for PSMattas
Puranjay Savar Mattas requested review from Evercatch Bot 2026-02-16 11:22:43 +00:00
Evercatch Bot approved these changes 2026-02-16 11:23:00 +00:00
Evercatch Bot left a comment
Member

Auto-Approved for PSMattas

Auto-Approved for PSMattas
Author
Owner

Manually Merged

Manually Merged
All checks were successful
PR Management Bot / pr-bot (pull_request) Successful in 14s
Required
Details
PR Title Checker / Validate PR Title Format (pull_request) Successful in 2s
Required
Details

Pull request closed

Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Evercatch/.profile#1