Skip to content

Sort action run jobs by JobID and Name with matrix examples#37046

Draft
bircni wants to merge 1 commit intogo-gitea:mainfrom
bircni:naturalsort-action-jobs
Draft

Sort action run jobs by JobID and Name with matrix examples#37046
bircni wants to merge 1 commit intogo-gitea:mainfrom
bircni:naturalsort-action-jobs

Conversation

@bircni
Copy link
Copy Markdown
Member

@bircni bircni commented Mar 30, 2026

Fix the sorting of jobs out of a matrix

Before

grafik

After

grafik

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 30, 2026
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Mar 30, 2026
@bircni bircni force-pushed the naturalsort-action-jobs branch from a531c68 to 9b82829 Compare March 30, 2026 18:02
Copy link
Copy Markdown
Contributor

@wxiaoguang wxiaoguang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will break the legacy "index-based" URLs if there is no full-migration.

Support legacy run/job index-based URLs and refactor migration 326 #37008

@GiteaBot GiteaBot added lgtm/blocked A maintainer has reservations with the PR and thus it cannot be merged and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 30, 2026
@bircni
Copy link
Copy Markdown
Member Author

bircni commented Mar 30, 2026

It will break the legacy "index-based" URLs if there is no full-migration.

Support legacy run/job index-based URLs and refactor migration 326 #37008

Oh yes you are right! Maybe when @Zettat123 is done we can discuss that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/blocked A maintainer has reservations with the PR and thus it cannot be merged modifies/go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants