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

Update config-cheat-sheet.en-us.md #8497

Merged
merged 6 commits into from
Oct 15, 2019
Merged

Update config-cheat-sheet.en-us.md #8497

merged 6 commits into from
Oct 15, 2019

Conversation

bkmgit
Copy link
Contributor

@bkmgit bkmgit commented Oct 14, 2019

Add more information on configuring URI hyperlink rendering for Markdown.
Partially addressing #8478

Add more information on configuring URI hyperlink rendering for Markdown.
@bkmgit bkmgit mentioned this pull request Oct 14, 2019
7 tasks
@@ -108,6 +108,8 @@ Values containing `#` or `;` must be quoted using `` ` `` or `"""`.
## Markdown (`markdown`)

- `ENABLE_HARD_LINE_BREAK`: **false**: Enable Markdown's hard line break extension.
- `CUSTOM_URL_SCHEMES`: : Use a comma separated list (http,https,ftp,git,svn) to indicate
URI hyperlinks that should be rendered in Markdown. The default is only http and https.
Copy link
Member

Choose a reason for hiding this comment

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

It's not URI but URL in this case. Also, http and https are always rendered no matter what this setting says.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should be part of the pr that does the custom URL schemes assuming we haven't merged that yet

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@guillep2k @zeripath Thanks for suggestions. Updated contents as suggested. Feel free to incorporate into pr for custom URL schemes.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Oct 14, 2019
Update description as suggested by @guillep2k
@codecov-io
Copy link

codecov-io commented Oct 14, 2019

Codecov Report

Merging #8497 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8497      +/-   ##
==========================================
- Coverage   41.72%   41.71%   -0.02%     
==========================================
  Files         512      512              
  Lines       66942    66942              
==========================================
- Hits        27930    27923       -7     
- Misses      35422    35433      +11     
+ Partials     3590     3586       -4
Impacted Files Coverage Δ
models/unit.go 62.16% <0%> (-5.41%) ⬇️
modules/task/migrate.go 24.35% <0%> (-3.85%) ⬇️
models/error.go 32.08% <0%> (-1.25%) ⬇️
models/repo_list.go 73.17% <0%> (-0.98%) ⬇️
routers/repo/view.go 40% <0%> (-0.96%) ⬇️
models/gpg_key.go 55.83% <0%> (-0.84%) ⬇️
models/repo.go 47.89% <0%> (+0.05%) ⬆️
modules/log/event.go 65.64% <0%> (+1.02%) ⬆️
modules/migrations/gitea.go 11.75% <0%> (+1.49%) ⬆️
modules/migrations/migrate.go 22.9% <0%> (+1.67%) ⬆️

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 366806d...3a0b401. 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 Oct 14, 2019
@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 14, 2019
@guillep2k
Copy link
Member

Build failing due to #8474 unrelated to this PR.

@zeripath zeripath added the type/docs This PR mainly updates/creates documentation label Oct 14, 2019
@zeripath zeripath added this to the 1.11.0 milestone Oct 14, 2019
@zeripath
Copy link
Contributor

Make lg-tm work

@zeripath zeripath merged commit ebe8ff7 into go-gitea:master Oct 15, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
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