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

os: format readme, fix markdown inside html #21286

Merged
merged 2 commits into from
Apr 15, 2024

Conversation

ttytm
Copy link
Member

@ttytm ttytm commented Apr 15, 2024

Auto-formats the os modules readme. Doing so it fixes an issue with markdown inside html tags.
The formatting changes match with what is used for the bigger part other markdown files in the codebase.

Following commonmarks recommended formatting for markdown inside html tags works here as a fix. Stackoverflow answer with link to commonmark example: https://stackoverflow.com/a/29378250

Current After formatting
Screenshot_20240415_084810 Screenshot_20240415_084800

Putting markdown immediately after a tag never works(at least as far as I know). It even breaks highlighting in the example below. It can work to have a tag immediately after markdown. The latter stopped working for vdoc after the recent fixes regarding indentation and escape characters. The recommended way is to keep a line-break before and after tags. Maybe following recommended commonmark formatting (which is applied by markdown formatters) is good enough as a fix now.

MD immediatly after opening tag MD followed by tag
Screenshot_20240415_090704 Screenshot_20240415_090711

@ttytm ttytm changed the title os: format readme to, fix markdown inside html os: format readme, fix markdown inside html Apr 15, 2024
@ttytm
Copy link
Member Author

ttytm commented Apr 15, 2024

In terms of a "fix" the PR will not be required anymore after #21287 (comment). If desired, the formatting here can still be done for the sake of formatting.

@spytheman spytheman merged commit e676245 into vlang:master Apr 15, 2024
4 checks passed
@ttytm ttytm deleted the os/format-readme branch April 15, 2024 12:18
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