Skip to content

GitAuto: Low Test Coverage: services/github/comments/get_comments.py#809

Closed
gitauto-for-dev[bot] wants to merge 1 commit intomainfrom
gitauto-wes/issue-807-20250510-225534
Closed

GitAuto: Low Test Coverage: services/github/comments/get_comments.py#809
gitauto-for-dev[bot] wants to merge 1 commit intomainfrom
gitauto-wes/issue-807-20250510-225534

Conversation

@gitauto-for-dev
Copy link
Copy Markdown
Contributor

Resolves #807

Test these changes locally

git fetch origin
git checkout gitauto-wes/issue-807-20250510-225534
git pull origin gitauto-wes/issue-807-20250510-225534

@gitauto-for-dev gitauto-for-dev Bot requested a review from hiroshinishio May 10, 2025 13:57
hiroshinishio added a commit that referenced this pull request Apr 19, 2026
The handler compared check suite app names (e.g. "CircleCI Checks") against
required status check context strings (e.g. "build_deploy_development_workflow"),
which never matched. Now compares by app_id available in both data sources.

Also: add MergeableState Literal type, improve log messages, add real-repo
test cases for website (PRs #809-#811) and Foxquilt with 50 total tests.
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.

Low Test Coverage: services/github/comments/get_comments.py

1 participant