|
|
b85d40f587
|
fix(pr merge): on 405, fetch and print PR merge status
|
2026-04-01 17:08:49 +00:00 |
|
|
|
f782bce0a3
|
fix(pr show): remove user and assignee fields
|
2026-04-01 17:08:49 +00:00 |
|
|
|
1feef0c5b6
|
fix(pr show): slim down output, remove nested repo/user objects
|
2026-04-01 17:08:49 +00:00 |
|
|
|
5725ac79bb
|
feat(pr): add show subcommand and refine merge output messages
- Add 'pr show' subcommand: GET /repos/{owner}/{repo}/pulls/{index}
- Change 200 success: 'merge success'
- Change 405 error: 'merge failed check the pr status'
|
2026-04-01 17:08:49 +00:00 |
|
|
|
a7700a5f15
|
fix(pr): for 405 errors, output API message directly
|
2026-04-01 17:08:49 +00:00 |
|
|
|
d764f30589
|
fix(pr): proper HTTP status code handling for merge command
Handle different HTTP status codes returned by merge API:
- 200: success (check merged field)
- 409: conflict (output error message)
- 404: not found
- 423: repo archived
- 405: empty response / try again later
- etc: generic error handling
|
2026-04-01 17:08:49 +00:00 |
|
|
|
77fa2f77bc
|
fix(pr): use /pulls instead of /pulls/pinned for list command
|
2026-04-01 17:08:49 +00:00 |
|
|
|
bd41addc7d
|
Resolve conflicts: keep pr command with show subcommand
|
2026-04-01 17:08:49 +00:00 |
|
|
|
3930adf362
|
fix: extract actual token from secret-mgr output
secret-mgr get-secret outputs 'Access token was successfully created: <token>'
so we need to extract just the token using awk '{print $NF}'
|
2026-04-01 17:08:02 +00:00 |
|
|
|
2d1d77f57a
|
Resolve conflicts: keep pr command (renamed from create-pr)
|
2026-04-01 17:08:02 +00:00 |
|
