Skip to content

Show workflow link#37070

Merged
wxiaoguang merged 9 commits intogo-gitea:mainfrom
bircni:reenable/workflow-link
Apr 1, 2026
Merged

Show workflow link#37070
wxiaoguang merged 9 commits intogo-gitea:mainfrom
bircni:reenable/workflow-link

Conversation

@bircni
Copy link
Copy Markdown
Member

@bircni bircni commented Apr 1, 2026

Thanks to #34538 this is super easy to add 😄

Superseeds #31906

Example

Bildschirmfoto 2026-04-01 um 11 41 07

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 1, 2026
@github-actions github-actions bot added modifies/templates This PR modifies the template files modifies/frontend labels Apr 1, 2026
@silverwind
Copy link
Copy Markdown
Member

silverwind commented Apr 1, 2026

I think #36721 is a better approach that matches GitHub's display, sorry it's been stalled a bit but I plan to pick it up again.

@bircni
Copy link
Copy Markdown
Member Author

bircni commented Apr 1, 2026

I think #36721 is a better approach that matches GitHub's display, sorry it's been stalled a bit but I plan to pick it up again.

The backbutton yes but the workflow file is also on GitHub available in the bottom and I like that better - I'd also suggest to remove the upper link in your pr then

@silverwind
Copy link
Copy Markdown
Member

silverwind commented Apr 1, 2026 •
edited
Loading

Ah I see you are adding this "Run details" on the bottom of the list. Does Github contain 2 links to the workflow file or just that one? If just one, then these PRs dont't conflict and I will remove the link in mine.

@bircni
Copy link
Copy Markdown
Member Author

bircni commented Apr 1, 2026

Does Github contain 2 links to the workflow file or just that one? If just one, then these PRs dont't conflict and I will remove the link in mine.

Only that one: https://github.com/bircni/actions-test-repo/actions/runs/23791719228
Bildschirmfoto 2026-04-01 um 11 50 59

silverwind
silverwind reviewed Apr 1, 2026
View reviewed changes
silverwind
silverwind reviewed Apr 1, 2026
View reviewed changes
@silverwind silverwind mentioned this pull request Apr 1, 2026
Draft
bircni
bircni commented Apr 1, 2026
View reviewed changes
@bircni
Apply suggestion from @bircni
491609e 
Signed-off-by: Nicolas <bircni@icloud.com>
@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 Apr 1, 2026
@wxiaoguang
fine tune layout
9d2f014 
# Conflicts:
#	web_src/js/components/RepoActionView.vue
@wxiaoguang
fix misalignment
83f9886
@wxiaoguang

This comment was marked as outdated.

Sign in to view
@bircni
Copy link
Copy Markdown
Member Author

bircni commented Apr 1, 2026

Bildschirmfoto 2026-04-01 um 13 00 54 This has now no space ...

@wxiaoguang
Copy link
Copy Markdown
Contributor

wxiaoguang commented Apr 1, 2026 •
edited
Loading

This has now no space ...

Found it, so there are new commits

image

wxiaoguang
wxiaoguang approved these changes Apr 1, 2026
View reviewed changes
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.

Should be good enough now.

New feature, simpler and clearer layout, less code

@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 Apr 1, 2026
@bircni
Copy link
Copy Markdown
Member Author

bircni commented Apr 1, 2026

Should be good enough now.

New feature, simpler and clearer layout, less code

yep looks good!
Thanks for the enhancements

@wxiaoguang wxiaoguang mentioned this pull request Apr 1, 2026
Closed
@wxiaoguang
Copy link
Copy Markdown
Contributor

wxiaoguang commented Apr 1, 2026 •
edited
Loading

Hmm, still a question: should we move the "Run Details" to the top?

Summary
Run Details
Jobs
Artifacts

The order is from their priority: a run manages its jobs.

OK, ignore me since the decision is from GitHub ....... although it looks strange to me.

@bircni
Copy link
Copy Markdown
Member Author

bircni commented Apr 1, 2026

@wxiaoguang be would could make it expandable? so open/close idk if this would be useful...

@wxiaoguang
Copy link
Copy Markdown
Contributor

wxiaoguang commented Apr 1, 2026

be would could make it expandable?

Which section does the "it" refer to? Or merge this as is and leave the improvements to the future.

@bircni
Copy link
Copy Markdown
Member Author

bircni commented Apr 1, 2026

yes lets merge this - I will try some stuff later on maybe

@bircni
Copy link
Copy Markdown
Member Author

bircni commented Apr 1, 2026

also when this is done @silverwind can finally continue - would like to see his stuff soon :-)

@wxiaoguang wxiaoguang enabled auto-merge (squash) April 1, 2026 11:35
@wxiaoguang wxiaoguang added the type/enhancement An improvement of existing functionality label Apr 1, 2026
@wxiaoguang wxiaoguang merged commit ca8c713 into go-gitea:main Apr 1, 2026
26 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Apr 1, 2026
silverwind added a commit to silverwind/gitea that referenced this pull request Apr 1, 2026
@silverwind
Merge remote-tracking branch 'origin/main' into darkmono
52658d0 
* origin/main:
  Update JS dependencies and misc tweaks (go-gitea#37064)
  Redirect to the only OAuth2 provider when no other login methods and fix various problems (go-gitea#36901)
  Show workflow link (go-gitea#37070)
  Remove leftover `webpackChunkName` comments from codeeditor (go-gitea#37062)
  Update Go dependencies (go-gitea#36781)
  Add webhook name field to improve webhook identification (go-gitea#37025) (go-gitea#37040)
  Upgrade `go-git` to v5.17.2 (go-gitea#37060)
  Replace Monaco with CodeMirror (go-gitea#36764)
  Update Combine method to treat warnings as failures and adjust tests (go-gitea#37048)
  Raise minimum Node.js version to 22.18.0 (go-gitea#37058)
  Update golangci-lint to v2.11.4 (go-gitea#37059)
  Upgrade `golang.org/x/image` to v0.38.0 (go-gitea#37054)

# Conflicts:
#	web_src/css/themes/theme-gitea-dark.css
zjjhot added a commit to zjjhot/gitea that referenced this pull request Apr 2, 2026
@zjjhot
Merge branch 'main' into mainzjj
0646014 
* main:
  Fix NuGet package upload error handling (go-gitea#37074)
  Desaturate dark theme background colors (go-gitea#37056)
  Update JS dependencies and misc tweaks (go-gitea#37064)
  Redirect to the only OAuth2 provider when no other login methods and fix various problems (go-gitea#36901)
  Show workflow link (go-gitea#37070)
  Remove leftover `webpackChunkName` comments from codeeditor (go-gitea#37062)
  Update Go dependencies (go-gitea#36781)
  Add webhook name field to improve webhook identification (go-gitea#37025) (go-gitea#37040)
  Upgrade `go-git` to v5.17.2 (go-gitea#37060)
  Replace Monaco with CodeMirror (go-gitea#36764)
  Update Combine method to treat warnings as failures and adjust tests (go-gitea#37048)
  Raise minimum Node.js version to 22.18.0 (go-gitea#37058)
  Update golangci-lint to v2.11.4 (go-gitea#37059)
  Upgrade `golang.org/x/image` to v0.38.0 (go-gitea#37054)
  Increase e2e test timeouts on CI to fix flaky tests (go-gitea#37053)
  Refactor "org teams" page and help new users to "add member" to an org (go-gitea#37051)
silverwind added a commit to silverwind/gitea that referenced this pull request Apr 2, 2026
@silverwind
Merge remote-tracking branch 'origin/main' into feedcut
77eeee7 
* origin/main: (192 commits)
  Fix NuGet package upload error handling (go-gitea#37074)
  Desaturate dark theme background colors (go-gitea#37056)
  Update JS dependencies and misc tweaks (go-gitea#37064)
  Redirect to the only OAuth2 provider when no other login methods and fix various problems (go-gitea#36901)
  Show workflow link (go-gitea#37070)
  Remove leftover `webpackChunkName` comments from codeeditor (go-gitea#37062)
  Update Go dependencies (go-gitea#36781)
  Add webhook name field to improve webhook identification (go-gitea#37025) (go-gitea#37040)
  Upgrade `go-git` to v5.17.2 (go-gitea#37060)
  Replace Monaco with CodeMirror (go-gitea#36764)
  Update Combine method to treat warnings as failures and adjust tests (go-gitea#37048)
  Raise minimum Node.js version to 22.18.0 (go-gitea#37058)
  Update golangci-lint to v2.11.4 (go-gitea#37059)
  Upgrade `golang.org/x/image` to v0.38.0 (go-gitea#37054)
  Increase e2e test timeouts on CI to fix flaky tests (go-gitea#37053)
  Refactor "org teams" page and help new users to "add member" to an org (go-gitea#37051)
  Refactor issue sidebar and fix various problems (go-gitea#37045)
  Add tests for pull request's content_version in API (go-gitea#37044)
  Enable concurrent vitest execution (go-gitea#36998)
  Fix theme discovery and Vite dev server in dev mode (go-gitea#37033)
  ...

# Conflicts:
#	templates/user/dashboard/feeds.tmpl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@silverwind silverwind silverwind approved these changes

@wxiaoguang wxiaoguang wxiaoguang approved these changes

Assignees

No one assigned

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/frontend modifies/templates This PR modifies the template files type/enhancement An improvement of existing functionality

Projects

None yet

Milestone

1.26.0

Development

Successfully merging this pull request may close these issues.

4 participants

@bircni @silverwind @wxiaoguang @GiteaBot