-
Notifications
You must be signed in to change notification settings - Fork 35
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
embedded HTML code creates issues for MkDocs and results in wrong display on wiki.rusefi.com #313
Comments
details and summary tags work fine in Wiki3, but you have to add markdown parameters - see Cranking.md for example. |
This is really thin ice and from my view won't work reliably without proper documentation and tooling! |
documentation and tooling is missing, #318 only fixed "tip of iceberg" |
Documentation now exists in https://wiki.rusefi.com/HOWTO-contribute-to-documentation/ |
Do you happen to know https://github.com/DavidAnson/markdownlint-cli2? my decision factors are driven by long-term benefits & reliability not only by efforts.
is unfortunately not an option to trust on because even an editor with only the best intent (like me) may introduce errors unintentionally due to lacking context, skill or requirements for a specific aspect. |
markdownlint doesn't come anywhere close to doing what we need without writing our own checks for it. I guess if somebody wants to do that, they can, but it seems to me like a poor use of developer time. |
Could you perhaps add what you know to be needed? |
did you check https://github.com/DavidAnson/markdownlint-cli2 as the tool I currently use? |
markdownlint only checks for presence of HTML. |
If you want to modify markdownlint to do that, you may. If not, let's close this issue. |
again #363 :( |
Fixed. Details in #268 |
example:
root cause:
fix:
use only markdown tags and remove embedded HTML code
affected markdown files
117 results - 8 files
The text was updated successfully, but these errors were encountered: