Skip to content

Conversation

wolfogre
Copy link
Member

@wolfogre wolfogre commented Nov 9, 2022

A simple refactor to reduce duplicate codes.

@wolfogre wolfogre added type/refactoring Existing code has been cleaned up. There should be no new functionality. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels Nov 9, 2022
@wolfogre wolfogre requested a review from lunny November 9, 2022 07:22
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Nov 9, 2022
@wolfogre wolfogre requested a review from KN4CK3R November 9, 2022 08:26
@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 Nov 9, 2022
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 10, 2022
@delvh
Copy link
Member

delvh commented Nov 10, 2022

Note

I didn't touch the meaningfulness of these error messages, I simply restored consistency a bit.

@lunny lunny merged commit 1d22911 into go-gitea:main Nov 10, 2022
@lunny lunny removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 10, 2022
zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 11, 2022
* giteaofficial/main:
  Revert unrelated changes for SMTP auth (go-gitea#21767)
  Init git module before database migration (go-gitea#21764)
  Extract updateSession function to reduce repetition (go-gitea#21735)
  Fix dashboard ignored system setting cache (go-gitea#21621)
  Add .dockerignore (go-gitea#21753)
fsologureng pushed a commit to fsologureng/gitea that referenced this pull request Nov 11, 2022
A simple refactor to reduce duplicate codes.

Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: zeripath <[email protected]>
Co-authored-by: delvh <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/refactoring Existing code has been cleaned up. There should be no new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants