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: Convert HTML to Markdown #5064

Merged
merged 5 commits into from
Feb 8, 2025

Conversation

wenzeslaus
Copy link
Member

This converts HTML markup in .md files to Markdown using the ./utils/grass_html2md.sh script.

This is committed with --no-verify. There are new or different Markdown linting rules broken and trailing spaces are not allowed.

How do we want to do it with the trailing spaces? Our pre-commit checks don't allow them, but they are used in Markdown extensively now to accommodate the various <br> used for formatting and it is used to mimic <dt>.

This converts HTML markup in .md files to Markdown using the ./utils/grass_html2md.sh script.
@echoix
Copy link
Member

echoix commented Feb 7, 2025

In https://github.com/OSGeo/grass/pull/4792/files#diff-0947e2727d6bad8cd0ac4122f5314bb5b04e337393075bc4b5ef143b17fcbd5bR75 (not merged yet), I noticed it too and did not enable. I thought I added the exclusion in pre-commit too... it didn't have other problems after that.

@echoix
Copy link
Member

echoix commented Feb 7, 2025

For the time of the conversion, even if it takes a week, ignore or disable the problematic pre-commit checks (for markdown if you want). It's quite easy to fix after the big 3-4 person effort for markdown docs is done

@wenzeslaus
Copy link
Member Author

The conversion from HTML to Markdown together with the automatic fixes removed a ton of warnings. I removed the relevant ignore and added three which did not occur with the HTML. I added .md to exclude for trailing spaces.

…but not terribly and the history is not that important for these two (one is a recently added mkdocs template and the other may be even unused).
@wenzeslaus wenzeslaus enabled auto-merge (squash) February 7, 2025 19:46
@github-actions github-actions bot added windows Microsoft Windows specific GUI wxGUI related CI Continuous integration vector Related to vector data processing raster Related to raster data processing temporal Related to temporal data processing database Related to database management libraries module docs general display imagery tests Related to Test Suite raster3d misc labels Feb 7, 2025
@wenzeslaus wenzeslaus merged commit a82a39f into OSGeo:main Feb 8, 2025
27 checks passed
@wenzeslaus wenzeslaus deleted the convert-html-to-markdown branch February 8, 2025 00:16
@github-actions github-actions bot added this to the 8.5.0 milestone Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration database Related to database management display docs general GUI wxGUI related imagery libraries misc module raster Related to raster data processing raster3d temporal Related to temporal data processing tests Related to Test Suite vector Related to vector data processing windows Microsoft Windows specific
Projects
Development

Successfully merging this pull request may close these issues.

3 participants