FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix locked issue crash + robustness changes by SableRaf · Pull Request #6308 · processing/processing · GitHub

Repository navigation

Fix locked issue crash + robustness changes - #6308

Merged
SableRaf merged 1 commit into
masterfrom
SableRaf-fix-workflow-crash-on-locked-issues-1
Jul 15, 2026
Merged

SableRaf merged 1 commit into
masterfrom
SableRaf-fix-workflow-crash-on-locked-issues-1

Conversation

SableRaf commented Jul 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Note

this needs testing against the actual repository.

This PR fixes: Error: Unhandled error: HttpError: Unable to create comment because issue is locked.

See action log: https://github.com/processing/processing/actions/runs/29001253630

Summary of changes:

  • Skips commenting/locking on threads that are already locked
  • Wraps per-thread processing in try/catch so one failure doesn't abort the whole run
  • Closes PRs via pulls.update instead of issues.update
  • Adds a concurrency group to prevent overlapping runs
  • Handles labels returned as plain strings, not just objects
  • Breaks out of the loop once a thread is newer than the cutoff (list is already sorted) instead of just continue
  • Builds close messages via array + join('\n') instead of a template literal (avoids leading-whitespace indentation in the posted comment)

> [!NOTE]
> this needs testing against the actual repository.

This PR fixes the following issue: `Error: Unhandled error: HttpError: Unable to create comment because issue is locked.`

See: https://github.com/processing/processing/actions/runs/29001253630

#### Summary of changes:

-   Skips commenting/locking on threads that are already locked (fixes the `HttpError: Unable to create comment because issue is locked` crash)
-   Wraps per-thread processing in try/catch so one failure doesn't abort the whole run
-   Closes PRs via `pulls.update` instead of `issues.update`
-   Adds a `concurrency` group to prevent overlapping runs
-   Handles labels returned as plain strings, not just objects
-   Breaks out of the loop once a thread is newer than the cutoff (list is already sorted) instead of just `continue`
-   Builds close messages via array + `join('\n')` instead of a template literal (avoids leading-whitespace indentation in the posted comment)
SableRaf marked this pull request as ready for review July 9, 2026 08:29
SableRaf requested a review from catilac July 9, 2026 08:29
SableRaf merged commit f480ef6 into master Jul 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL