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.

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

Attempt to improve docs (yet again) #18687

Merged
merged 6 commits into from
Feb 10, 2022
Merged

Conversation

zeripath
Copy link
Contributor

@zeripath zeripath commented Feb 9, 2022

Attempt to make the passthrough docs clearer.
Add some more faq entries

Signed-off-by: Andrew Thornton art27@cantab.net

Attempt to make the passthrough docs clearer.
Add some more faq entries

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added the type/docs This PR mainly updates/creates documentation label Feb 9, 2022
@zeripath zeripath added this to the 1.17.0 milestone Feb 9, 2022
@zeripath
Copy link
Contributor Author

zeripath commented Feb 9, 2022

Please if there are changes - either push directly or make ```suggestions

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 9, 2022
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/help/faq.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
docs/content/doc/installation/with-docker.en-us.md Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

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

❗ Current head d254ff8 differs from pull request most recent head d38dc5a. Consider uploading reports for the commit d38dc5a to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #18687   +/-   ##
=======================================
  Coverage        ?   46.65%           
=======================================
  Files           ?      846           
  Lines           ?   121367           
  Branches        ?        0           
=======================================
  Hits            ?    56623           
  Misses          ?    57864           
  Partials        ?     6880           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f76608...d38dc5a. Read the comment docs.

@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 Feb 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 Feb 10, 2022
@zeripath
Copy link
Contributor Author

make lgtm

@zeripath zeripath merged commit 165ab17 into go-gitea:main Feb 10, 2022
@zeripath zeripath deleted the minor-docs-changes branch February 10, 2022 19:20
zjjhot added a commit to zjjhot/gitea that referenced this pull request Feb 11, 2022
* giteaofficial/main:
  Prevent double encoding of branch names in delete branch (go-gitea#18714)
  [skip ci] Updated translations via Crowdin
  Attempt to improve docs (yet again) (go-gitea#18687)
  Make the proformas clearer that we need DEBUG logs (go-gitea#18700)
  Update SSH Server crypto settings (go-gitea#18697)
  Fix bug for version update hint (go-gitea#18701)
Chianina pushed a commit to Chianina/gitea that referenced this pull request Mar 28, 2022
* Attempt to improve docs (yet again)

Attempt to make the passthrough docs clearer.
Add some more faq entries

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Apply suggestions from code review

* Apply suggestions from code review

Co-authored-by: Gusted <williamzijl7@hotmail.com>

* Update docs/content/doc/installation/with-docker.en-us.md

* Update docs/content/doc/installation/with-docker.en-us.md

Co-authored-by: Gusted <williamzijl7@hotmail.com>
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
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.

6 participants