Skip to content

Conversation

@luqven
Copy link
Owner

@luqven luqven commented Dec 29, 2025

Summary

  • Add api/checks.rs module with GitHub Checks API integration
  • fetch_check_status() - get CI status for a commit SHA
  • fetch_mergeable_status() - get PR mergeable state
  • CheckStatus / CheckState types for status representation
  • Proper error handling for rate limits and API errors

Tests

14 tests including:

  • Unit tests for parsing check run responses
  • Async/mock tests for API calls
  • Error handling tests (rate limits, timeouts)

Dependencies

  • Adds tempfile dev dependency for upcoming legend file tests

Part 2 of 5 in the status-command stack

Stacked PR Chain: status-command

PR Title Merges Into
#30 Add head_sha/html_url accessors to PullRequest -
#31 👉Add GitHub Checks API module #30
#32 Add status module with core logic and rendering #31
#33 Add status subcommand and log --status flag #32
#34 Add status command documentation #33

Add api/checks.rs with:
- fetch_check_status() - get CI status for a commit SHA
- fetch_mergeable_status() - get PR mergeable state
- CheckStatus/CheckState types for status representation
- Proper error handling for rate limits and API errors
- 14 tests including unit tests and async/mock tests

Also adds tempfile dev dependency for upcoming legend file tests.
@luqven
Copy link
Owner Author

luqven commented Dec 29, 2025

Landed via #34

@luqven luqven closed this Dec 29, 2025
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.

2 participants