Skip to content

Conversation

@nwx-natew
Copy link
Contributor

Changes to be committed:
modified: docs/auditor/10.8/configuration/microsoft365/exchangeonline/modernauth/manifest.md
modified: docs/auditor/10.8/configuration/microsoft365/microsoftentraid/permissions/modernauth/manifest.md
modified: docs/auditor/10.8/configuration/microsoft365/sharepointonline/permissions/modernauth/manifest.md
modified: docs/auditor/10.8/configuration/microsoft365/teams/permissions/modernauth/manifest.md

Per MD040: Fenced code blocks should have a language specified Updated in all four articles to specify "json" as the language for the code blocks containing JSON manifests. Added benefit of syntax highlighting in rendered markdown.

 Changes to be committed:
	modified:   docs/auditor/10.8/configuration/microsoft365/exchangeonline/modernauth/manifest.md
	modified:   docs/auditor/10.8/configuration/microsoft365/microsoftentraid/permissions/modernauth/manifest.md
	modified:   docs/auditor/10.8/configuration/microsoft365/sharepointonline/permissions/modernauth/manifest.md
	modified:   docs/auditor/10.8/configuration/microsoft365/teams/permissions/modernauth/manifest.md

Per [MD040](https://github.com/DavidAnson/markdownlint/blob/v0.38.0/doc/md040.md.md): Fenced code blocks should have a language specified
Updated in all four articles to specify "json" as the language for the code blocks containing JSON manifests.
Added benefit of syntax highlighting in rendered markdown.
@Denis-A-Netwrix
Copy link

Denis-A-Netwrix commented Sep 30, 2025

It seems json text has some unexpected spaces which can not be handled my markdown engine. So, I see highlighted red texts:
image
Could you update the formatting and make sure that it's shown correctly in preview mode?

@nwx-natew
Copy link
Contributor Author

nwx-natew commented Oct 7, 2025

Hey Denis. I'm afraid I am not able to reproduce this when I run it locally. I see correct syntax highlighting like so:
image
This is how it looks to me for all four data sources.

@Denis-A-Netwrix
Copy link

Hey Denis. I'm afraid I am not able to reproduce this when I run it locally. I see correct syntax highlighting like so: image This is how it looks to me for all four data sources.

I see it in github preview mode
https://github.com/netwrix/docs/blob/eb7972e65416ba1b21377b3cc29ecc838866d725/docs/auditor/10.8/configuration/microsoft365/exchangeonline/modernauth/manifest.md

@nwx-natew
Copy link
Contributor Author

I updated to use jsonc instead. This way GitHub shows it as syntactically correct but won't confuse users when they read the documentation pages themselves.

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.

2 participants