Skip to content

Conversation

@luqven
Copy link
Owner

@luqven luqven commented Dec 29, 2025

Summary

  • Add head_sha() accessor for commit SHA (needed for check status lookups)
  • Add raw_title() accessor for unformatted title display
  • Add html_url() method to convert API URLs to web URLs
  • Handle both github.com and enterprise GitHub URLs

Tests

5 unit tests added for the new accessors.


Part 1 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

These accessors are needed for the upcoming status command:
- head_sha() returns the commit SHA for check status lookups
- raw_title() returns unformatted title for display
- html_url() converts API URL to web URL for display

Handles both github.com and enterprise GitHub URLs.
@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