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

change markdown to "Markdown" in.... #624

Merged
merged 1 commit into from Sep 29, 2019
Merged

change markdown to "Markdown" in.... #624

merged 1 commit into from Sep 29, 2019

Conversation

leela52452
Copy link
Contributor

...A universal document converter. Can be used to convert markdown to a wide range of other formats.
see https://en.wikipedia.org/wiki/Markdown

...A universal document converter. Can be used to convert markdown to a wide range of other formats.
see https://en.wikipedia.org/wiki/Markdown
@gedakc
Copy link
Collaborator

gedakc commented Sep 5, 2019

If the word "Markdown" is to be capitalized when referred to in English sentences, then have you checked for other usages of "markdown" that should also be capitalized?

@leela52452
Copy link
Contributor Author

the comment on weblate source made sense to me and i proposed change. i haven't checked other instances. i will look into it after few weeks.

@gedakc
Copy link
Collaborator

gedakc commented Sep 9, 2019

the comment on weblate source made sense to me and i proposed change. i haven't checked other instances. i will look into it after few weeks.

Based on the source link provided I think it was highlighting the whitespace in the string, not whether "Markdown" was capitalized or not.

@gedakc gedakc mentioned this pull request Sep 9, 2019
@gedakc gedakc added this to the 0.10.0 milestone Sep 27, 2019
@gedakc
Copy link
Collaborator

gedakc commented Sep 29, 2019

Since "Markdown" seems to be referred to with a capital "M", I will merge this PR with the develop branch.

@gedakc gedakc merged commit decb60c into olivierkes:develop Sep 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants