Skip to content

fix: catch more expected getPullRequestForCommit errors#217

Merged
cjquines merged 1 commit intomainfrom
cj/catch-get-pr-for-commit
Mar 20, 2026
Merged

fix: catch more expected getPullRequestForCommit errors#217
cjquines merged 1 commit intomainfrom
cj/catch-get-pr-for-commit

Conversation

@cjquines
Copy link
Member

@cjquines cjquines commented Mar 20, 2026

should close #213

@cjquines cjquines requested a review from meorphis March 20, 2026 14:57
Copy link
Member

@meorphis meorphis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

worth logging before returning here?

@cjquines
Copy link
Member Author

the error should be logged by default logger anyway

@cjquines cjquines merged commit 30a19a7 into main Mar 20, 2026
2 checks passed
@cjquines cjquines deleted the cj/catch-get-pr-for-commit branch March 20, 2026 15:01
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.

403 Resource not accessible by integration

2 participants