Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Changelog 1.19.2 #24365

Merged
merged 9 commits into from Apr 27, 2023
Merged

Changelog 1.19.2 #24365

merged 9 commits into from Apr 27, 2023

Conversation

jolheiser
Copy link
Member

No description provided.

@jolheiser
chore: changelog 1.19.2
797a824 
Signed-off-by: jolheiser <john.olheiser@gmail.com>
@jolheiser
chore: add security change
fe4cb92 
Signed-off-by: jolheiser <john.olheiser@gmail.com>
@jolheiser jolheiser added this to the 1.19.2 milestone Apr 27, 2023
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 27, 2023
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 27, 2023
@jolheiser jolheiser added the type/docs This PR mainly updates/creates documentation label Apr 27, 2023
techknowlogick
techknowlogick reviewed Apr 27, 2023
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
techknowlogick
techknowlogick reviewed Apr 27, 2023
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
@jolheiser
review: tech
806a87b 
Signed-off-by: jolheiser <john.olheiser@gmail.com>
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 27, 2023
@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 Apr 27, 2023
@wxiaoguang
Copy link
Contributor

Don't set meta theme-color by default (#24340) #24346 ?

@jolheiser
Copy link
Member Author

jolheiser commented Apr 27, 2023 •
edited

Don't set meta theme-color by default (#24340) #24346 ?

Strange that doesn't show up in the generation. 🤔

EDIT: I think there are more missing as well...will need to look

EDIT 2: Apparently the GH search API is off, I switched the changelog tool to use the issue list API and committed the change in f8c31bb

@lunny
Copy link
Member

lunny commented Apr 27, 2023

#24369 merged

pat-s
pat-s reviewed Apr 27, 2023
View reviewed changes
Copy link
Member

@pat-s pat-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This Gitea version is the first to add "proper" support for the latest version of redis-cluster (due to the update of the redis go library and #23869 (which was in 1.19.1). This is a big step for high availability. Maybe worth mentioning it as a "feature" or at least "enhancement"?

@jolheiser
Copy link
Member Author

This Gitea version is the first to add "proper" support for the latest version of redis-cluster (due to the update of the redis go library and #23869 (which was in 1.19.1). This is a big step for high availability. Maybe worth mentioning it as a "feature" or at least "enhancement"?

I think this would be great to add to the blog post and gh release notes! I'll let you know when there's a blog pad and we can add it. 🥳

@wxiaoguang
Copy link
Contributor

wxiaoguang commented Apr 27, 2023 •
edited

Do we need to figure out this ? My LFS client doesn't work

#24362 (comment)

@delvh delvh added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Apr 27, 2023
@jolheiser
Copy link
Member Author

Blocked by #24382

@delvh delvh added status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR and removed reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. labels Apr 27, 2023
@yardenshoham
Copy link
Member

Merged

@yardenshoham yardenshoham added reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. and removed status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR labels Apr 27, 2023
@lunny lunny merged commit 4bc6bfb into go-gitea:release/v1.19 Apr 27, 2023
1 of 2 checks passed
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Apr 27, 2023
@6543 6543 added the frontport/main This PR is not targeting the main branch. Still, its changes should also be added there label Apr 28, 2023
yardenshoham pushed a commit to yardenshoham/gitea that referenced this pull request Apr 28, 2023
@jolheiser @yardenshoham
Changelog 1.19.2 (go-gitea#24365)
a06bc4a 
Add changelog for 1.19.2

Signed-off-by: jolheiser <john.olheiser@gmail.com>
@yardenshoham yardenshoham mentioned this pull request Apr 28, 2023
Merged
@yardenshoham yardenshoham added the frontport/done The frontport PR has been created label Apr 28, 2023
silverwind pushed a commit that referenced this pull request Apr 28, 2023
@yardenshoham @jolheiser @GiteaBot
Changelog 1.19.2 (#24365) (#24403)
3843252 
Frontport #24365

Add changelog for 1.19.2

Signed-off-by: jolheiser <john.olheiser@gmail.com>
Co-authored-by: John Olheiser <john.olheiser@gmail.com>
Co-authored-by: Giteabot <teabot@gitea.io>
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Jul 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pat-s pat-s pat-s left review comments

@lunny lunny lunny approved these changes

@techknowlogick techknowlogick techknowlogick approved these changes

@delvh delvh delvh approved these changes

Assignees
No one assigned
Labels
frontport/done The frontport PR has been created frontport/main This PR is not targeting the main branch. Still, its changes should also be added there lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. type/docs This PR mainly updates/creates documentation
Projects
None yet
Milestone
1.19.2
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@jolheiser @wxiaoguang @lunny @yardenshoham @techknowlogick @pat-s @delvh @GiteaBot @6543