Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented Feb 2, 2023

caused by #22680

pushPayload.Ref and prPayload.PullRequest.Base.Ref have the format like refs/heads/<branch_name>, so we need to trim the prefix before comparing.

@lunny lunny added the type/bug label Feb 2, 2023
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Feb 2, 2023
@wolfogre wolfogre added this to the 1.19.0 milestone Feb 2, 2023
@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 Feb 2, 2023
@lunny lunny merged commit 368d436 into go-gitea:main Feb 2, 2023
@lunny lunny deleted the lunny/fix_workflow_match branch February 2, 2023 12:40
@lunny lunny added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Feb 2, 2023
zjjhot added a commit to zjjhot/gitea that referenced this pull request Feb 5, 2023
* giteaoffical/main: (23 commits)
  Add Cargo package registry (go-gitea#21888)
  Add new captcha: cloudflare turnstile (go-gitea#22369)
  add default user visibility to cli command "admin user create" (go-gitea#22750)
  Show all projects, not just repo projects and open/closed projects  (go-gitea#22640)
  Remove ONLY_SHOW_RELEVANT_REPOS setting (go-gitea#21962)
  Escape path for the file list (go-gitea#22741)
  Repositories: by default disable all units except code and pulls on forks (go-gitea#22541)
  Fix color of tertiary button on dark theme (go-gitea#22739)
  Refactor git command package to improve security and maintainability (go-gitea#22678)
  Improve trace logging for pulls and processes (go-gitea#22633)
  Remove 'primary' class from tab counter labels (go-gitea#22687)
  Use native error checking with `exec.ErrDot` (go-gitea#22735)
  update to build with go1.20 (go-gitea#22732)
  Add missed reverse proxy authentication documentation (go-gitea#22250)
  Update button is shown when a Pull Request is marked WIP - Issue go-gitea#21740 (go-gitea#22683)
  Do not overwrite empty DefaultBranch (go-gitea#22708)
  Improve error report when user passes a private key (go-gitea#22726)
  Add some comments for recent code (go-gitea#22725)
  Fix actions workflow branches match bug (go-gitea#22724)
  Fix group filter for ldap source sync (go-gitea#22506)
  ...
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
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. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants