-
-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
docs: 📝Update README to improve Markdown linting and structure #18498
base: main
Are you sure you want to change the base?
Conversation
Fixed in |
The current one with more padding feels more balanced 🤔 |
Fixed in 475ea5d |
Done. |
I'll let others take a look before, to be honest the added markdownlint comments are quite noisy. |
They do not interfere with anything and do not bite |
If we want to introduce |
Look, I'm not planning to add CI for linting now, I want it to be at least this way now, I'll look at the vitepres support later Let this PR stay. |
Tbh I don't really see the point of changing the markup and looking the same anyways. Sorry to be a bit harsh here, but I'd appreciate if the PRs were addressing actual bugs or issues, rather than subjective / own-opinion / styling changes, especially for first time contributions. |
Well, you suffocated.. |
review |
Description
Improved and simplified centering of elements, added exclusion of some rules for
markdownlint
, allowed to use HTML only where it is really necessary (tags for centering elements) and nowhere else, this will prevent the use of HTML instead of native Markdown, which will positively affect the readability of the code