Skip to content

Add richer PR details to the changes pane#57

Merged
penso merged 1 commit intomainfrom
side-pane-pr-details
Mar 13, 2026
Merged

Add richer PR details to the changes pane#57
penso merged 1 commit intomainfrom
side-pane-pr-details

Conversation

@penso
Copy link
Copy Markdown
Owner

@penso penso commented Mar 13, 2026

Summary

  • add a shared PR summary card above the changes list and reuse its status presentation in the sidebar hover UI
  • show richer GitHub PR state, including checks, review state, mergeability, loading feedback, and expandable check details
  • fetch PR details for worktrees in parallel and precompute check ordering/counts so the UI render path avoids blocking I/O and per-render sorting

Testing

  • just format
  • just lint
  • cargo +nightly-2025-11-30 test -p arbor-gui prioritized_pr_checks_show_failures_before_pending_before_successes
  • cargo +nightly-2025-11-30 test -p arbor-gui in_progress_checks_remain_pending_without_conclusion

@codspeed-hq
Copy link
Copy Markdown
Contributor

codspeed-hq bot commented Mar 13, 2026

Merging this PR will not alter performance

✅ 21 untouched benchmarks


Comparing side-pane-pr-details (218d064) with main (7204e25)

Open in CodSpeed

@penso penso merged commit 5a99ccc into main Mar 13, 2026
12 checks passed
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.

1 participant