Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Repository.getFilesChanged to fix Actions paths and paths-ignore filter #23920

Merged
merged 3 commits into from
Apr 7, 2023

Conversation

ChristopherHX
Copy link
Contributor

@ChristopherHX ChristopherHX commented Apr 4, 2023

Remove the misbehaving function and call Repository.GetFilesChangedBetween instead.

Fixes #23919


TODO test this Repository.getFilesChanged seems to be only used by Gitea Actions, but a similar function already exists

Update I tested this change and the issue is gone.

@ChristopherHX ChristopherHX marked this pull request as ready for review April 4, 2023 21:03
@silverwind silverwind added this to the 1.20.0 milestone Apr 5, 2023
Copy link
Member

@wolfogre wolfogre left a comment

Choose a reason for hiding this comment

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

LGTM. getFilesChanged is an old implementation, while GetFilesChangedBetween has a more robust approach.

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 6, 2023
@wolfogre wolfogre changed the title Remove Repository.getFilesChanged to fix Actions paths and paths Remove Repository.getFilesChanged to fix Actions paths and paths filter Apr 6, 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 Apr 6, 2023
@lunny lunny added reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. outdated/backport/v1.19 This PR should be backported to Gitea 1.19 labels Apr 6, 2023
@delvh delvh changed the title Remove Repository.getFilesChanged to fix Actions paths and paths filter Remove Repository.getFilesChanged to fix Actions paths and paths filter Apr 6, 2023
@ChristopherHX ChristopherHX changed the title Remove Repository.getFilesChanged to fix Actions paths and paths filter Remove Repository.getFilesChanged to fix Actions paths and paths-ignore filter Apr 6, 2023
@lunny lunny enabled auto-merge (squash) April 7, 2023 02:56
@lunny lunny disabled auto-merge April 7, 2023 06:41
@lunny lunny merged commit 17623bb into go-gitea:main Apr 7, 2023
@lunny lunny removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Apr 7, 2023
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Apr 7, 2023
…-ignore` filter (go-gitea#23920)

Remove the misbehaving function and call
Repository.GetFilesChangedBetween instead.

Fixes go-gitea#23919

---

~~_TODO_ test this~~ `Repository.getFilesChanged` seems to be only used
by Gitea Actions, but a similar function already exists

**Update** I tested this change and the issue is gone.
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Apr 7, 2023
zjjhot added a commit to zjjhot/gitea that referenced this pull request Apr 7, 2023
* upstream/main:
  Clean template/helper.go (go-gitea#23922)
  Remove `Repository.getFilesChanged` to fix Actions `paths` and `paths-ignore` filter (go-gitea#23920)
  Hardcode path to docker images (go-gitea#23955)
  Title can be empty when creating tag only (go-gitea#23917)
  Actions: Use default branch as ref when a branch/tag delete occurs (go-gitea#23910)
  Refactor authors dropdown (send get request from frontend to avoid long wait time) (go-gitea#23890)
  [skip ci] Updated translations via Crowdin
  Merge `push to create`, `open PR from push`, and `push options` docs articles into one (go-gitea#23744)
  Delete deleted release attachments immediately from storage (go-gitea#23913)
  More specific and unique feed name for NuGet install command template. (go-gitea#23889)
  Treat PRs with agit flow as fork PRs when triggering actions. (go-gitea#23884)
  Use graceful editorconfig loader to reduce errors when loading malformed editorconfigs (go-gitea#21257)
  Remove -v from vulncheck (go-gitea#23953)
  Improve permission check of packages (go-gitea#23879)
  Adjust some documentations titles (go-gitea#23941)
@ChristopherHX ChristopherHX deleted the patch-1 branch April 7, 2023 20:00
lunny pushed a commit that referenced this pull request Apr 8, 2023
…-ignore` filter (#23920) (#23969)

Backport #23920 by @ChristopherHX

Remove the misbehaving function and call
Repository.GetFilesChangedBetween instead.

Fixes #23919

---

~~_TODO_ test this~~ `Repository.getFilesChanged` seems to be only used
by Gitea Actions, but a similar function already exists

**Update** I tested this change and the issue is gone.

Co-authored-by: ChristopherHX <[email protected]>
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. outdated/backport/v1.19 This PR should be backported to Gitea 1.19 type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Repository.getFilesChanged(start, end) contains the empty path filename and breaks Actions paths filter
6 participants