Skip to content

Simplify context ref name#33267

Merged
wxiaoguang merged 2 commits into
go-gitea:mainfrom
wxiaoguang:refactor-branch-dropdown
Jan 15, 2025
Merged

Simplify context ref name#33267
wxiaoguang merged 2 commits into
go-gitea:mainfrom
wxiaoguang:refactor-branch-dropdown

Conversation

@wxiaoguang

@wxiaoguang wxiaoguang commented Jan 14, 2025

Copy link
Copy Markdown
Contributor

By using the new design, the code could be simplified and clearer than before.

  • IsViewBranch: replaced by RefFullName.IsBranch
  • IsViewTag: replaced by RefFullName.IsTag

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 14, 2025
Comment thread routers/web/repo/commit.go
@wxiaoguang wxiaoguang mentioned this pull request Jan 14, 2025
@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 Jan 15, 2025
@lunny lunny added this to the 1.24.0 milestone Jan 15, 2025
@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 Jan 15, 2025
@wxiaoguang wxiaoguang merged commit fcd0962 into go-gitea:main Jan 15, 2025
@wxiaoguang wxiaoguang deleted the refactor-branch-dropdown branch January 15, 2025 03:15
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jan 15, 2025
* giteaofficial/main:
  Simplify context ref name (go-gitea#33267)
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Apr 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants