Skip to content

fix(actions): show run index in run view and fix summary graph height#38165

Merged
bircni merged 6 commits into
go-gitea:mainfrom
bircni:feat-actions-run-index-summary-layout
Jun 22, 2026
Merged

fix(actions): show run index in run view and fix summary graph height#38165
bircni merged 6 commits into
go-gitea:mainfrom
bircni:feat-actions-run-index-summary-layout

Conversation

@bircni

@bircni bircni commented Jun 18, 2026

Copy link
Copy Markdown
Member
  • Display the per-repository run number as #N next to the run title in the run view, matching the runs list and GitHub
  • Add the run Index to the run view API response (and the devtest mock) to support that
  • Restore the summary panel's flex: 1 so the workflow graph fills the right-column height even when a run has no job summaries
  • Keep the job-summary section content-sized so it doesn't compete with the graph for height
  • Gate the devtest mock job summaries to a subset of runs so the devtest page also exercises the no-summary layout
image

- Add the per-repository run number (Index) to the run view response and
  render it as "#N" next to the run title, matching the runs list and GitHub
- Restore the summary panel's flex:1 so the workflow graph fills the right
  column height even when a run has no job summaries
- Keep the job-summary section content-sized so it doesn't steal graph height
- Gate the devtest mock job summaries to a couple of runs so the devtest page
  also exercises the no-summary layout
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jun 18, 2026
@bircni bircni added this to the 1.27.0 milestone Jun 18, 2026
@bircni bircni requested a review from silverwind June 18, 2026 15:54
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jun 18, 2026
@bircni bircni requested a review from delvh June 19, 2026 08:37
@bircni bircni requested review from TheFox0x7 and Zettat123 June 21, 2026 14:11
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jun 22, 2026
@bircni bircni added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Jun 22, 2026
@bircni bircni enabled auto-merge (squash) June 22, 2026 05:53
@bircni bircni merged commit 649cb6f into go-gitea:main Jun 22, 2026
23 checks passed
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Jun 22, 2026
@bircni bircni deleted the feat-actions-run-index-summary-layout branch June 22, 2026 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants