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

doc: change Russian readme extension to .md #217

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

UnkwUsr
Copy link

@UnkwUsr UnkwUsr commented Aug 4, 2024

I see English readme already uses .md extension, which makes nice style drawing on github site. While Russian readme still uses .txt extension

@UnkwUsr UnkwUsr marked this pull request as draft August 5, 2024 00:17
@UnkwUsr UnkwUsr marked this pull request as ready for review August 5, 2024 01:27
@UnkwUsr
Copy link
Author

UnkwUsr commented Aug 5, 2024

Задача оказалась сложнее чем я думал. Дело в том, что в markdown параграфом считается блок текста не разделённый целой отдельной пустой строкой. Просмотрел весь файл 2 раза, исправил всё что нашёл. Конечно, здесь требуется ещё много работы по добавлению красоты вроде inline-codeblocks

@bol-van
Copy link
Owner

bol-van commented Aug 5, 2024

там надо все проходить от и до
это работа не на раз два

Copy link

@Hi-Angel Hi-Angel left a comment

Choose a reason for hiding this comment

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

О, лайк, я как раз хотел тоже заняться конвертацией в markdown, хорошо что в PR'ы заглянул) Я правда думал сначала создать issue чтобы убедиться что мэйнтейнер согласен, но он согласен)

@@ -53,6 +53,7 @@ For english version refer to docs/readme.eng.txt
Существует и более продвинутая магия, направленная на преодоление DPI на пакетном уровне.

Choose a reason for hiding this comment

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

Этот коммит явно должен быть частью предыдущего, потому что в предыдущем получается сломанный markdown документ.

@@ -54,8 +54,8 @@ For english version refer to docs/readme.eng.txt

Choose a reason for hiding this comment

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

Этот коммит аналогично.

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.

3 participants