21356 commits

This branch

This branch

All branches

Author	SHA1	Message	Date
Gusted	8fdc0a7a6c	feat: combine review requests comments ... - Combine review requests comments similairy how labels comments are combined. If review requests comments were made within 60 seconds of each other they will be grouped. - Integration and unit test added. - Resolves #2774	2024-10-25 22:57:32 +02:00
Gusted	459ab11a8a	fix: use buffered iterate for debian searchpackages ... - The driver being used for PostgreSQL doesn't handle interleaved queries (you start a query, read some rows and start another query while you didn't finish that query yet), this is the case with using `.Iterate` from XORM. - Switch to a variant of what exist in the current codebase of `db.Iterate`, which is a simple buffered iteration and doesn't keep queries open, which allow other database operations to happen. - Unit test added. This doesn't cover that postgres does not error on this case, as this is not run with a postgres database. - Resolves #5696	2024-10-25 19:33:19 +02:00
Otto	00cc563388	Merge pull request 'Do not change stroke size in the loading animation' (#5506) from kytta/forgejo:forgejo-loading-stroke into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5506 Reviewed-by: Otto <otto@codeberg.org>	2024-10-25 17:24:16 +00:00
0ko	37dca47ab2	chore(release-notes-assistant): update internationalization label (#5698) ... There was an attempt to rename this label to a shorter one, but release-notes-assistant.sh relies on the old name. Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5698 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>	2024-10-25 14:36:22 +00:00
Gusted	5af8a55f6a	Merge pull request 'tests(e2e): skip browser downloads' (#5692) from fnetx/playwright-nodownload into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5692 Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-25 03:30:48 +00:00
Otto	8b7410f35c	Merge pull request 'chore: output playwright directly to std{out,err}' (#5685) from gusted/forgejo-improve-e2e-test into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5685 Reviewed-by: Otto <otto@codeberg.org>	2024-10-24 20:44:19 +00:00
Otto Richter	511b213db2	tests(e2e): skip browser downloads ... The browser versions are updated weekly in the cache, see https://code.forgejo.org/forgejo/playwright-image-builder When they change between the runs, they are refreshed for each CI run (see https://codeberg.org/forgejo/forgejo/actions/runs/35651/jobs/3#jobstep-7-19 for an example). Updating the browsers weekly is good enough, so skip the explicit update check here. Related playwright documentation: https://playwright.dev/docs/library#browser-downloads	2024-10-24 22:42:21 +02:00
Earl Warren	b30203bc48	Merge pull request 'fix: make branch protection work for new branches' (#5688) from gusted/forgejo-branch-protection-new-branch into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5688 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>	2024-10-24 20:20:49 +00:00
Gusted	f5e025917f	fix: make branch protection work for new branches ... - If `GetAffectedFiles` is called for a push with an empty oldCommitID, then set the oldCommitID to the empty tree. This will effictively diff all the changes included in the push, which is the expected behavior for branches. - Integration test added. - Resolves #5683 - Port of gitea#31778 but implemented differently.	2024-10-24 18:44:58 +02:00
Michael Kriese	ec4a0e1b6e	fix: reset history.scrollRestoration if set to manual and no issue anchor in url	2024-10-24 15:25:51 +02:00
Gusted	78d243c304	chore: output playwright directly to std{out,err} ... Instead of letting playwright do the full test suite and then print the output and error, direct the output to std{our,err} for a faster developing loop. This also makes the output colored.	2024-10-24 15:06:19 +02:00
Gusted	aa86e94853	Merge pull request 'Update dependency postcss-nesting to v13.0.1 (forgejo)' (#5678) from renovate/forgejo-postcss into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5678 Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-24 12:53:47 +00:00
Codeberg Translate	f72567ee14	i18n: update of translations from Codeberg Translate (#5583) ... Co-authored-by: SomeTr <SomeTr@users.noreply.translate.codeberg.org> Co-authored-by: eldyj <eldyj@users.noreply.translate.codeberg.org> Co-authored-by: Benedikt Straub <Nordfriese@users.noreply.translate.codeberg.org> Co-authored-by: tkbremnes <tkbremnes@users.noreply.translate.codeberg.org> Co-authored-by: Gusted <postmaster@gusted.xyz> Co-authored-by: meskobalazs <meskobalazs@users.noreply.translate.codeberg.org> Co-authored-by: div72 <div72@users.noreply.translate.codeberg.org> Co-authored-by: xtex <xtexchooser@duck.com> Co-authored-by: kmpm <kmpm@users.noreply.translate.codeberg.org> Co-authored-by: Fnurkla <Fnurkla@users.noreply.translate.codeberg.org> Co-authored-by: Juno Takano <jutty@users.noreply.translate.codeberg.org> Co-authored-by: Fjuro <fjuro@alius.cz> Co-authored-by: buhtz <buhtz@users.noreply.translate.codeberg.org> Co-authored-by: 0ko <0ko@users.noreply.translate.codeberg.org> Co-authored-by: Dirk <Dirk@users.noreply.translate.codeberg.org> Co-authored-by: Outbreak2096 <Outbreak2096@users.noreply.translate.codeberg.org> Co-authored-by: Atul_Eterno <Atul_Eterno@users.noreply.translate.codeberg.org> Co-authored-by: artnay <artnay@users.noreply.translate.codeberg.org> Co-authored-by: earl-warren <earl-warren@users.noreply.translate.codeberg.org> Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5583 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org> Co-authored-by: Codeberg Translate <translate@noreply.codeberg.org> Co-committed-by: Codeberg Translate <translate@noreply.codeberg.org>	2024-10-24 04:37:43 +00:00
Renovate Bot	840e68c187	Update dependency postcss-nesting to v13.0.1	2024-10-24 00:02:48 +00:00
forgejo-renovate-action	0f99a0e3c0	Merge pull request 'Update dependency eslint-plugin-playwright to v2 (forgejo)' (#5669) from renovate/forgejo-eslint-plugin-playwright-2.x into forgejo	2024-10-23 15:01:37 +00:00
Michael Kriese	412d16f236	Merge pull request 'chore: remove eslint v9 restriction in renovate' (#5668) from gusted/forgejo-eslint-renovate into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5668 Reviewed-by: Michael Kriese <michael.kriese@gmx.de>	2024-10-23 14:45:02 +00:00
Gusted	94e4f8648b	chore: add extra playwright rules	2024-10-23 16:22:25 +02:00
Renovate Bot	cdbbdacb7d	Update dependency eslint-plugin-playwright to v2	2024-10-23 14:05:20 +00:00
Gusted	18a959dc8c	chore: remove eslint v9 restriction in renovate ... This restriction can be removed with 7ad83fce40	2024-10-23 16:02:27 +02:00
Gusted	7a29ab4d20	Merge pull request 'chore: move to Eslint flat config' (#5662) from gusted/forgejo-flat-config into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5662 Reviewed-by: Michael Kriese <michael.kriese@gmx.de>	2024-10-23 14:00:28 +00:00
Gusted	ee75b784d1	Merge pull request 'Update module github.com/jhillyerd/enmime to v2 (forgejo)' (#5665) from renovate/forgejo-github.com-jhillyerd-enmime-2.x into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5665	2024-10-23 13:40:04 +00:00
Gusted	3a4bc7cdd1	Merge pull request 'feat: use combo markdown editor for milestone description' (#5657) from gusted/forgejo-milestone-comboeditor into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5657 Reviewed-by: Otto <otto@codeberg.org>	2024-10-23 13:38:35 +00:00
Gusted	7ad83fce40	chore: move to Eslint flat config ... Make the big move to Eslint flat config format. The outcome of Eslint still should be the same, but some things has changed: - `eslint-plugin-github` is dropped, flat configs have been out for a while and most eslint plugins support it, but for no reason and no activity in sight this plugin is likely not going to support flat config for a while and to avoid other plugins not being able to update (as they are requiring flat configs) drop the github rules. - Nested configs don't work properly and are unified into the root eslint config, this unification did cause some conflicts and thats why the `import-x` is in a seperate 'group' to exclude targeting Vue files. - The `eslint-plugin-i` is deprecated and `esplint-plugin-import-x` is its succesor which has better support for flat configs, the same rules are still applied. The majority of the flat config was generated by `@eslint/migrate-config` tool.	2024-10-23 15:28:43 +02:00
Gusted	e3eaae4b56	chore: update license path	2024-10-23 15:09:12 +02:00
forgejo-renovate-action	e4de08b258	Merge pull request 'Update dependency eslint-plugin-unicorn to v56 (forgejo)' (#5663) from renovate/forgejo-eslint-plugin-unicorn-56.x into forgejo	2024-10-23 07:29:18 +00:00
Gusted	20a233e770	Merge pull request 'Update dependency postcss-nesting to v13 (forgejo)' (#5664) from renovate/forgejo-postcss-nesting-13.x into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5664 Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-23 06:48:29 +00:00
Gusted	f63f02045e	chore: add new lint rules	2024-10-23 08:10:18 +02:00
Renovate Bot	b597725fa0	Update module github.com/jhillyerd/enmime to v2	2024-10-23 06:04:11 +00:00
Renovate Bot	b4253ec7e7	Update dependency postcss-nesting to v13	2024-10-23 06:03:35 +00:00
Renovate Bot	1ec3e638a5	Update dependency eslint-plugin-unicorn to v56	2024-10-23 06:03:20 +00:00
Gusted	4c6587d6a0	feat: use combo markdown editor for milestone description ... - Use the combo markdown editor for the milestone description. The milestone description is rendered in markdown, so it makes sense to use a 'markdown-aware' editor. This also includes the option to use monospace font. - Resolves #5649	2024-10-22 19:58:44 +02:00
Earl Warren	ac5736005b	Merge pull request 'Update module github.com/prometheus/client_golang to v1.20.5 (forgejo)' (#5240) from renovate/forgejo-github.com-prometheus-client_golang-1.x into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5240 Reviewed-by: Michael Kriese <michael.kriese@gmx.de>	2024-10-22 17:38:33 +00:00
Earl Warren	c1a08156f8	Merge pull request '[gitea] week 2024-43 cherry pick (gitea/main -> forgejo)' (#5621) from algernon/wcp/2024-43 into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5621 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org> Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-22 17:37:03 +00:00
Earl Warren	334975c445	Merge pull request 'link to security policy in security.txt' (#5651) from fnetx/securitytxt into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5651 Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-22 16:50:19 +00:00
Earl Warren	fac25238d4	Merge pull request 'fix: don't show truncated comments in RSS/Atom feeds' (#5653) from gusted/forgejo-truncated-comment-rss into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5653 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>	2024-10-22 16:48:52 +00:00
Gergely Nagy	93dccfffe1	chore(release-notes): notes for week 2024-43 weekly cherry pick ... Signed-off-by: Gergely Nagy <forgejo@gergo.csillger.hu>	2024-10-22 18:14:27 +02:00
Gusted	6f1c1cdb04	Merge pull request 'fix: typo on releases for source code downloads' (#5652) from viceice/fix/typo into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5652 Reviewed-by: Otto <otto@codeberg.org> Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-22 14:02:49 +00:00
Michael Kriese	02f8fad54d	fix: typo on releases for source code downloads ... Closes #5648	2024-10-22 15:18:31 +02:00
Gusted	f4a7132a89	fix: don't show truncated comments in RSS/Atom feeds ... - When a truncated comment is detected in the RSS/Atom feeds, fetch the comment from the database and use the original content. - Added integration test. - Resolves #5650	2024-10-22 15:15:09 +02:00
Otto Richter	d06f1c6856	link to security policy in security.txt	2024-10-22 14:28:17 +02:00
Gusted	2626f4ee3b	Merge pull request 'conf: Improve delete_old_actions description' (#5644) from xenrox/forgejo:conf-improvement into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5644 Reviewed-by: Otto <otto@codeberg.org> Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-22 11:05:38 +00:00
Earl Warren	aa4d098844	chore(license): Update module github.com/prometheus/client_golang to v1.20.5 (forgejo)	2024-10-22 07:48:36 +02:00
Earl Warren	884b21efd5	Merge pull request 'chore(release-notes): no need to specify they are draft' (#5646) from earl-warren/forgejo:wip-rna into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5646 Reviewed-by: Michael Kriese <michael.kriese@gmx.de>	2024-10-22 05:44:12 +00:00
Earl Warren	f586944db3	chore(release-notes): no need to specify they are draft ... Since they are written to the milestone that is still open, there is no risk of confusing them with final release notes. Such a distinction is more relevant when in the context of a release notes file committed to the repository.	2024-10-22 06:54:27 +02:00
Gusted	e45c304b31	Merge pull request 'Revert "add gap between branch dropdown and PR button"' (#5640) from fnetx/dropdown-glitch into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5640 Reviewed-by: Gusted <gusted@noreply.codeberg.org> Reviewed-by: Panagiotis "Ivory" Vasilopoulos <git@n0toose.net>	2024-10-21 20:43:54 +00:00
Thorben Günther	c2f7aca9da	conf: Improve delete_old_actions description ... This matches the text in the web interface. The old description can be easily mistaken for handling "Forgejo Actions".	2024-10-21 22:31:50 +02:00
Gusted	6c0698c7de	Merge pull request 'fix: make syncronize tags to database handle annoted tags' (#5641) from gusted/forgejo-sync-tags into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5641 Reviewed-by: Otto <otto@codeberg.org>	2024-10-21 18:45:21 +00:00
0ko	3e1d5cc447	feat(ui): better activity messages for cases of private profiles (#5638) ... * added a message for the case where the user's profile is private but the Public activity is not hidden The activity is still hidden anyway because the profile is private, but previously the message would say: `Your activity is visible to everyone, except for interactions in private spaces...` which I would consider as a flaw of the original implementation. Now it will say: `Your activity is only visible to you and the instance administrators because your profile is private...` * started showing the message for admins that the activity they see should remain private in the case where the whole profile is private, not just the activity tab. Previously it would say: This activity is visible to everyone, but as an administrator you can also see interactions in private spaces.` which I would also consider as a flaw of the original implementation. Now it will say: `This activity is visible to you because you're an administrator, but the user wants it to remain private.` * added test cases * bumped up the number of our GPL-licensed files Preview For both screenshots, Forgejo would previously display misinformation. Change 1: User viewing their private profile, but activity isn't configured as hidden https://codeberg.org/attachments/6659c80c-15dd-48be-a379-db737fd1dd5e Change 2: Admin viewing user's private profile https://codeberg.org/attachments/220da57f-b658-4474-9ad2-049e8438a0af Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5638 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org> Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-21 14:39:29 +00:00
Gusted	f298bf125a	Merge pull request 'fix: use ValidateEmail as binding across web forms' (#5158) from solomonv/consolidate-email-validation into forgejo ... Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5158 Reviewed-by: Gusted <gusted@noreply.codeberg.org>	2024-10-21 14:31:32 +00:00
Gusted	6da194fae8	fix: make syncronize tags to database handle annoted tags ... - When an admin wants syncronize tags in the Git data to the database via the admin dashboard all annoted tags loses their title. This was caused because the code didn't correctly handle annoted tags. Annoted tags have their own objectID to store the annoted message, unlike 'normal' tags which point to the commitID. While the function was being run for annoted tags, the code thought it found a mismatch in the objectIDs, because the stored version was actually correct which pointed to the commitID but the code found the objectID of the annoted tag. - Make `SyncReleasesWithTags` corectly handle annoted tags. - Added unit and integration tests. - Resolves #5628	2024-10-21 16:21:07 +02:00