Backport #34334 by @lunny
The `ci.HeadGitRepo` was opened and closed in the function
`ParseCompareInfo` but reused in the function `PrepareCompareDiff`.
---------
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Backport #34286 by @lunny
Fix#34224
The previous implementation in #33744 will get the pushed commits
changed files. But it's not always right when push a merged commit. This
PR reverted the logic in #33744 and will always get the PR's changed
files and get code owners.
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Backport #34244 by kemzeb
Resolves#34218.
In the recent past, the default wiki branch was made to be changeable.
This change reflects this.
Co-authored-by: Kemal Zebari <60799661+kemzeb@users.noreply.github.com>
Backport #34153 by @wkelly17
This commit replaces the hardcoded string "code" in the clone panel
button with the i18n local for repo.code.
Co-authored-by: Will Kelly <67284402+wkelly17@users.noreply.github.com>
Co-authored-by: silverwind <me@silverwind.io>
Fixes: https://pkg.go.dev/vuln/GO-2025-3563
Because of a bug in go, we can not remove the minor version in go.mod
without `make tidy` introducing a potentially harmful `toolchain`
directive, so just bump the go version here. From what I gather, this
will be fixed in go 1.25.
Backport #34100 by @ManInDark
Fixes#34099.
Resolved by checking the `key-cert.pub` location alongside the
previously configured location. In case a certificate is already found,
this won't change anything, but if there is one in `key-cert.pub` but
not in `key_cert`, it'll use that one now.
Co-authored-by: ManInDark <61268856+ManInDark@users.noreply.github.com>
Backport #34094 by @lunny
When visit commit list, it would update the user avatar even if id = 0,
which was unnecessary operations. This PR returned default avatar for
the git only user avatar rendering who's user id is zero.
```log
database duration=0.0005s db.sql="UPDATE `user` SET `avatar` = ?, `updated_unix` = ? WHERE `id`=?"
database duration=0.0007s db.sql="UPDATE `user` SET `avatar` = ?, `updated_unix` = ? WHERE `id`=?"
...
```
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Backport #34053
When a team have no code unit permission of a repository, the member of
the team should not view activity contributors, recent commits and code
frequrency.
---------
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Backport #34061 by eeyrjmr
The doctor storage check reconstructs the lfs oid by producing a string
where the path separator is stripped
ab/dc/efg -> abdcefg. Windows however uses a backslash and thus the
ReplaceAll call doesn't produce the correct oid resulting in all lfs
objects being classed as orphaned.
This PR allows this to be more OS agnostic.
Closes#34039
Co-authored-by: JonRB <4564448+eeyrjmr@users.noreply.github.com>
Fix#33490
Backport #33744
It will only read the changed file on the pushed commits but not all the
files of this PR.
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Backport #34035 by charles7668
Close#34022 , #33550
This error message always appears when using the `check-attr` command,
even though it works correctly.
The issue occurs when the stdin writer is closed, so I added a special
case to handle and check the error message when the exit code is 1.
Co-authored-by: charles <30816317+charles7668@users.noreply.github.com>
