Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Rename config option [camo].Allways to [camo].Always #32097

Merged
merged 1 commit into from
Sep 21, 2024
Merged

Rename config option [camo].Allways to [camo].Always #32097

merged 1 commit into from
Sep 21, 2024

Conversation

lunny
Copy link
Member

@lunny lunny commented Sep 21, 2024 •
edited
Loading

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 21, 2024
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 21, 2024
@lunny lunny added docs-update-needed The document needs to be updated synchronously and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 21, 2024
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Sep 21, 2024
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 21, 2024
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 21, 2024
@delvh delvh added the pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! label Sep 21, 2024
@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 Sep 21, 2024
@lafriks lafriks merged commit 8e2dd5d into go-gitea:main Sep 21, 2024
26 checks passed
@GiteaBot GiteaBot added this to the 1.24.0 milestone Sep 21, 2024
@lunny lunny deleted the lunny/correct_camo_allways branch September 21, 2024 19:13
@lunny lunny modified the milestones: 1.24.0, 1.23.0 Sep 21, 2024
zjjhot added a commit to zjjhot/gitea that referenced this pull request Sep 23, 2024
@zjjhot
Merge remote-tracking branch 'giteaofficial/main'
521030e 
* giteaofficial/main:
  [skip ci] Updated licenses and gitignores
  Fix rename branch permission bug (go-gitea#32066)
  Fix artifact v4 upload above 8MB (go-gitea#31664)
  [skip ci] Updated translations via Crowdin
  Add bin to Composer Metadata (go-gitea#32099)
  Fix wrong last modify time (go-gitea#32102)
  Fix upload maven pacakge parallelly (go-gitea#31851)
  Repo Activity: count new issues that were closed (go-gitea#31776)
  Count typescript files as frontend for labeling (go-gitea#32088)
  Use camo.Always instead of camo.Allways (go-gitea#32097)
@wxiaoguang wxiaoguang changed the title Use camo.Always instead of camo.Allways Rename config option [camo].Allways to [camo].Always Dec 17, 2024
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Dec 20, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers

@techknowlogick techknowlogick techknowlogick approved these changes

@delvh delvh delvh approved these changes

Assignees
No one assigned
Labels
docs-update-needed The document needs to be updated synchronously lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/go Pull requests that update Go code pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
1.23.0
Development

Successfully merging this pull request may close these issues.

Typo inside Camo#Allways
5 participants
@lunny @techknowlogick @delvh @lafriks @GiteaBot