-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Fix potential assignee query for repo #18994
Conversation
@zeripath I think you should check this. |
Time to add a test for this function? 😅 |
If someone would like to assist me there, I'm not yet familiar with the test system for Gitea ... |
@fnetX https://github.com/fnetX/gitea/pull/3 The code should be relatively self-explanatory. The "mocked" database files are loaded from |
Codecov Report
@@ Coverage Diff @@
## main #18994 +/- ##
=======================================
Coverage ? 46.59%
=======================================
Files ? 854
Lines ? 122819
Branches ? 0
=======================================
Hits ? 57227
Misses ? 58676
Partials ? 6916
Continue to review full report at Codecov.
|
@lunny has been merged. |
* Fix potential assignee query for repo * Add tests for `GetRepoAssignees` - As per #18994 (comment) Co-authored-by: Gusted <[email protected]>
make lgtm work |
* giteaofficial/main: Fix EasyMDE error when input Enter (go-gitea#19004) Fix update hint bug (go-gitea#18996) Fix the editor height in review box (go-gitea#19003) Add a "admin user generate-access-token" subcommand (go-gitea#17722) Fix potential assignee query for repo (go-gitea#18994) Add config option to disable "Update branch by rebase" (go-gitea#18745) Update `go-enry` to v2.8.0 (go-gitea#18993) homebrew updates via cron ignore missing comment for user notifications (go-gitea#18954) allow overwrite artifacts for github releases (go-gitea#18987) fix & refactor (go-gitea#18973)
* Fix potential assignee query for repo * Add tests for `GetRepoAssignees` - As per go-gitea#18994 (comment) Co-authored-by: 6543 <[email protected]> Co-authored-by: Gusted <[email protected]> Co-authored-by: Lunny Xiao <[email protected]>
Fix #18992