Copilot commented on code in PR #14290: URL: https://github.com/apache/cloudstack/pull/14290#discussion_r4158489306
########## .github/linters/.markdown-lint.yml: ########## @@ -69,9 +69,6 @@ MD034: false # MD036/no-emphasis-as-heading Emphasis used instead of a heading MD036: false -# MD037/no-space-in-emphasis Spaces inside emphasis markers -MD037: false - # MD040/fenced-code-language Fenced code blocks should have a language specified Review Comment: Enabling MD037 still leaves `tools/devcloud4/README.md:32` with `* Cloudstack Management Server *`, which matches the rule's invalid spaced-emphasis form. The markdownlint hook runs on all `.md` files and the ignore list excludes only `CHANGES.md`, so the pre-commit job will fail until that remaining occurrence is corrected. ########## tools/devcloud4/advanced/README.md: ########## @@ -46,7 +46,7 @@ vagrant up ``` -*** Common issues: *** +*** Common issues: Review Comment: Removing the closing delimiter leaves this label as literal `***` text, while the opening delimiter still contains an inner space. Keep both delimiters and remove the inner spaces so the label remains emphasized and conforms to MD037. This issue also appears on line 62 of the same file. ########## tools/devcloud4/basic/README.md: ########## @@ -36,7 +36,7 @@ vagrant up ``` -*** Common issues: *** +*** Common issues: Review Comment: Removing the closing delimiter leaves this label as literal `***` text, while the opening delimiter still contains an inner space. Keep both delimiters and remove the inner spaces so the label remains emphasized and conforms to MD037. This issue also appears on line 52 of the same file. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
