Skip to content

Conversation

@wxiaoguang
Copy link
Contributor

Drop the minimatch dependency, use our own glob compiler.

Fix #35473

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 12, 2025
@wxiaoguang wxiaoguang added this to the 1.25.0 milestone Sep 12, 2025
@wxiaoguang wxiaoguang force-pushed the fix-glob-match branch 2 times, most recently from 1fab533 to d445b67 Compare September 12, 2025 13:33
@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 Sep 12, 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 Sep 13, 2025
@wxiaoguang wxiaoguang merged commit 325e059 into go-gitea:main Sep 13, 2025
26 checks passed
@wxiaoguang wxiaoguang deleted the fix-glob-match branch September 13, 2025 03:53
zjjhot added a commit to zjjhot/gitea that referenced this pull request Sep 13, 2025
* giteaofficial/main:
  Fix different behavior in status check pattern matching with double stars (go-gitea#35474)
  Replace webpack with rspack (go-gitea#35460)
  Don't store repo archives on `gitea dump` (go-gitea#35467)
  Fix SSH signing key path will be displayed in the pull request UI (go-gitea#35381)
  [skip ci] Updated translations via Crowdin
  Update image name in integration README (go-gitea#35465)
@silverwind
Copy link
Member

silverwind commented Sep 13, 2025

FYI: https://github.com/go-gitea/gitea/security/code-scanning/144, maybe a false-positive. It seems most of these code scanning results are bogus.

@wxiaoguang
Copy link
Contributor Author

FYI: Double escaping or unescaping, maybe a false-positive. It seems most of these code scanning results are bogus.

False-positive. That code is only used for the special test data from Golang

@wxiaoguang
Copy link
Contributor Author

FYI: Double escaping or unescaping, maybe a false-positive. It seems most of these code scanning results are bogus.

False-positive. That code is only used for the special test data from Golang

To fix: fd02f8b

silverwind added a commit to ChristopherHX/gitea that referenced this pull request Sep 16, 2025
* origin/main:
  Clean up npm dependencies (go-gitea#35484)
  Update eslint to v9 (go-gitea#35485)
  Revert the rspack change (go-gitea#35482)
  Replace gobwas/glob package (go-gitea#35478)
  Fix various typos in codebase (go-gitea#35480)
  Fix different behavior in status check pattern matching with double stars (go-gitea#35474)
  Replace webpack with rspack (go-gitea#35460)
  Don't store repo archives on `gitea dump` (go-gitea#35467)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/dependencies modifies/frontend type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Different behavior in status check pattern matching with **

5 participants