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

Markdown Linter - Compliance #5

Closed
pocelka opened this issue Oct 10, 2020 · 2 comments
Closed

Markdown Linter - Compliance #5

pocelka opened this issue Oct 10, 2020 · 2 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@pocelka
Copy link

pocelka commented Oct 10, 2020

Hello,

Would it be possible to make Markdown output complaint with markdown linter? It currently produces:

@ogobrecht ogobrecht added enhancement New feature or request question Further information is requested labels Oct 10, 2020
@ogobrecht
Copy link
Owner

Hi,

thank you for the hint. There is also often MD033.

Let me explain, why I am doing it against common Markdown rules:

MD003 - Mixed heading styles

This is because SQL*Plus can be problematic when using the hash sign as the first character in a line. Therefore I use setext style headers in the Markdown comments. Unfortunately there are only two levels of headers with setext. If you need more levels you need to use ATX style headers in your code docs. I am personally prefer not to have more then two levels of headers in the docs - for me I do not get those MD003 warnings.

MD012 - Multiple consecutive blank lines

I use this to structure a little bit the generated code to be more readable in the pure text form. Two blank lines means a new section starts and inside this section I generate only single blank lines.

MD033 - Inline HTML

I use this to add a table of contents at the top of the documents and use HTML code to build the headers.

What can we do?

I don't want to change this for the mentioned reasons. To be compliant with the markdown linter I can add a hint at the top of the generated documents to disable these three rules for the Markdown linter.

Hope this is ok for you.

Best regards
Ottmar

@ogobrecht
Copy link
Owner

Hi,

version 0.6.2 is published on https://www.npmjs.com/

Best regards
Ottmar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants