ade41f3ftests: add coverage for models migration helpers (#18162)
They were previously not covered at all, either by integration tests or unit tests.
This PR also fixes a bug where the `num_comments` field was incorrectly set to include all types of comments.
It sets num_closed_issues: 0 as default in milestone unit test fixtures. If they are not set, Incr("num_closed_issues") will be a noop because the field is null.
Aravinth Manivannan
authored at
GitHub
comitted at
496acbe9Require codereview to have content (#18156)
- Report a validityError when the codeReview have no comment.
- Resolves #18151
- RefactorGusted
authored at
GitHub
comitted at
948949f4chore(lint): use golangci-lint to call revive and misspell checker. (#18145)
replace revive and misspell with golangci-lint
Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
Bo-Yi Wu
authored at
GitHub
comitted at
766cb481Update owners for 2022 (#18155)
6543
authored at
GitHub
comitted at
e61b390dUnify and simplify TrN for i18n (#18141)
Refer: https://github.com/go-gitea/gitea/pull/18135#issuecomment-1003246099
Now we have a unique and simple `TrN`, and make the fix of PR #18135 also use the better `TrN` logic.wxiaoguang
authored at
GitHub
comitted at
88da7a71Use correct user when determining max repo limits for error messages (#18153)
- Use the correct user(`owner` instead of `ctx.User`) to get the maxCreationLimit.Gusted
authored at
GitHub
comitted at
5d4ee44eAdd singuliere to MAINTAINERS (#18148)
Signed-off-by: singuliere <singuliere@autistici.org>singuliere
authored at
GitHub
comitted at
7db2f110Add API to get issue/pull comments and events (timeline) (#17403)
* Add API to get issue/pull comments and events (timeline)
Adds an API to get both comments and events in one endpoint with all required data.
Closes go-gitea/gitea#13250
* Fix swagger
* Don't show code comments (use review api instead)
* fmt
* Fix comment
* Time -> TrackedTime
* Use var directly
* Add logger
* Fix lint
* Fix test
* Add comments
* fmt
* [test] get issue directly by ID
* Update test
* Add description for changed refs
* Fix build issues + lint
* Fix build
* Use string enums
* Update swagger
* Support `page` and `limit` params
* fmt + swagger
* Use global slices
Co-authored-by: zeripath <art27@cantab.net>
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>qwerty287
authored at
GitHub
comitted at
549fd03cUpgrade certmagic from v0.14.1 to v0.15.2 (#18138)
Lunny Xiao
authored at
GitHub
comitted at
e9c9a35aUpgrade certmagic from v0.14.1 to v0.15.2 (#18138)
Lunny Xiao
authored at
GitHub
comitted at
385dc6a9Allow admin to associate missing LFS objects for repositories (#18143)
This PR reworked the Find pointer files feature in Settings -> LFS page.
When a LFS object is missing from database but exists in LFS content store, admin can associate it to the repository by clicking the Associate button.
This PR is not perfect (because the LFS module itself should be improved too), it's just a nice-to-have feature to help users recover their LFS repositories (eg: database was lost / table was truncated)wxiaoguang
authored at
GitHub
comitted at
1a4e2bfctests: s/GITEA_UNIT_TESTS_VERBOSE/GITEA_UNIT_TESTS_LOG_SQL/ (#18142)
The GITEA_UNIT_TESTS_VERBOSE variable is an undocumented variable
introduced in 2017 (see 1028ef2defd94a64f2433b07fe5d93681864cebb)
whose sole purpose has been to log SQL statements when running unit
tests.
It is renamed for clarity and a warning is displayed for backward
compatibility for people and scripts that know about it.
The documentation is updated to reflect this change.singuliere
authored at
GitHub
comitted at
fb2dc956services/repository: fix ListUnadoptedRepositories incorrect total count (#17865)
The total count returned by ListUnadoptedRepositories is incorrectly
calculated.
The code snippet within ListUnadoptedRepositories used to verify
unadopted repositories is repeated three times in the function. It is
moved in the checkUnadoptedRepositories function and a unit test is
added to verify it works as expected.
A unit test is added to verify the total count returned by
ListUnadoptedRepositories is as expected.
Signed-off-by: singuliere <singuliere@autistici.org>singuliere
authored at
GitHub
comitted at
c99b8efbImprove document for developers: Windows CGO, unit test option (#18140)
wxiaoguang
authored at
GitHub
comitted at
431e482eReset the conflicted files list in testpatch (#18139)
Although #18004 will seriously reduce the likelihood of finding
conflicts in the first place - one bug was introduced whereby the
conflicted files status was not being reset properly. This leads to
conflicted PRs remaining conflicted when the conflict has been resolved.
Signed-off-by: Andrew Thornton <art27@cantab.net>zeripath
authored at
GitHub
comitted at
7eb380baUse correct translation key (#18135)
- Resolves #18122
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>Gusted
authored at
GitHub
comitted at
ffaebb1a[Frontport] Changelog v1.15.9 (#18136)
* Changelog ## [1.15.9](https://github.com/go-gitea/gitea/releases/tag/v1.15.9) - 2021-12-30
* docs: bump version6543
authored at
GitHub
comitted at
67d7ad68Fix wrong redirect on org labels (#18128)
KN4CK3R
authored at
GitHub
comitted at
626b1079Fix performance regression when user has many organization (#18125)
Lunny Xiao
authored at
GitHub
comitted at
e0b41f8cdocs: add various ways to install from package (#18120)
zh-cn
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>Tony
authored at
GitHub
comitted at
658a636aRemoved unused method. (#18129)
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>KN4CK3R
authored at
GitHub
comitted at