Skip to content

Comments

Fix doc formatting issues#2007

Merged
flanakin merged 5 commits intodevfrom
flanakin/doc-formatting
Feb 24, 2026
Merged

Fix doc formatting issues#2007
flanakin merged 5 commits intodevfrom
flanakin/doc-formatting

Conversation

@flanakin
Copy link
Collaborator

🛠️ Description

Fix documentation formatting issues across docs-mslearn:

  • Add .markdownlint.json config to disable single-h1, line-length, and no-inline-html rules at the folder level (instead of per-file HTML comments)
  • Remove all per-file <!-- markdownlint-disable --> comments for MD025
  • Expand <!-- prettier-ignore-start/end --> blocks to wrap enclosing lists at column 0 (fixes Prettier collapsing multi-line MS Learn blockquotes inside list items)
  • Convert fragile markdownlint-disable-next-line comments (broken by Prettier inserting blank lines) to file-level markdownlint-disable
  • Clean up trailing blank lines and minor formatting inconsistencies

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests

🙋‍♀️ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

  • ❎ Log not needed (small/internal change)

📖 Did you update documentation?

  • ❎ Docs not needed (small/internal change)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes documentation formatting issues across the docs-mslearn folder by standardizing markdownlint and Prettier configuration.

Changes:

  • Adds .markdownlint.json to disable single-h1, line-length, and no-inline-html rules at folder level
  • Removes 150+ per-file <!-- markdownlint-disable-next-line MD025 --> comments (now handled by central config)
  • Wraps MS Learn blockquotes with <!-- prettier-ignore-start/end --> to prevent Prettier from collapsing multi-line content inside lists
  • Minor whitespace and formatting cleanups

Reviewed changes

Copilot reviewed 112 out of 112 changed files in this pull request and generated no comments.

File Description
.markdownlint.json New central configuration file disabling MS Learn-incompatible rules
All .md files Removed redundant MD025 comments, added prettier-ignore blocks around blockquotes in lists, minor whitespace fixes

@flanakin flanakin added this to the v14 milestone Feb 23, 2026
- Update all Power BI report export requirement tables
- Update FinOps hubs configure-scopes and data-processing docs
- Realign markdown tables for wider version column

🤖 Generated with [Claude Code](https://claude.ai/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@flanakin
Copy link
Collaborator Author

🤖 [AI][Claude] PR Update Summary

Addressed: 1 thread(s)

  • ✅ Implemented: 1

Added FOCUS 1.2-preview to supported version references across 9 files:

  • 7 Power BI report export requirement tables
  • FinOps hubs configure-scopes (supported versions list + export instructions)
  • FinOps hubs data-processing (supported versions list)

@flanakin flanakin merged commit 08bf628 into dev Feb 24, 2026
3 checks passed
@flanakin flanakin deleted the flanakin/doc-formatting branch February 24, 2026 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Review 👀 PR that is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants