Skip to content

Conversation

bwenrich
Copy link
Contributor

@bwenrich bwenrich commented Oct 9, 2022

Updates the Config Cheat Sheet for the ALLWAYS option in Camo proxy.

  • Clarifies the behavior for true (both HTTP and HTTPS is proxied) vs false (only HTTP is proxied)
  • Minor grammar and typo improvements

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Oct 10, 2022
@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 Oct 10, 2022
@a1012112796 a1012112796 added the type/docs This PR mainly updates/creates documentation label Oct 10, 2022
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@b59b0ca). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #21386   +/-   ##
=======================================
  Coverage        ?   47.44%           
=======================================
  Files           ?     1020           
  Lines           ?   139007           
  Branches        ?        0           
=======================================
  Hits            ?    65950           
  Misses          ?    65063           
  Partials        ?     7994           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@wxiaoguang wxiaoguang merged commit ea3c6c6 into go-gitea:main Oct 11, 2022
@bwenrich bwenrich deleted the camo-doc branch October 11, 2022 16:37
@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. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants