Skip to content

Fix : Remove markdownlint exclusion for collections/*.collection.md #1341#1372

Open
PratikWayase wants to merge 1 commit intomicrosoft:mainfrom
PratikWayase:fix/markdownlint-collections
Open

Fix : Remove markdownlint exclusion for collections/*.collection.md #1341#1372
PratikWayase wants to merge 1 commit intomicrosoft:mainfrom
PratikWayase:fix/markdownlint-collections

Conversation

@PratikWayase
Copy link
Copy Markdown
Contributor


Pull Request

Description

Removed the exclusion for collections/*.collection.md from markdownlint configuration and fixed all related lint violations across collection files.

Changes include:

  • Added top-level headings to all collection files (MD041)
  • Removed multiple consecutive blank lines (MD012)
  • Replaced em dashes with compliant alternatives
  • Standardized formatting and ensured newline consistency
  • Updated Validate-MarkdownFrontmatter.ps1 to align with updated linting rules

Related Issue(s)

Closes #1341


Type of Change

Select all that apply:

Code & Documentation:

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update

Infrastructure & Configuration:

  • GitHub Actions workflow
  • Linting configuration (markdown, PowerShell, etc.)
  • Security configuration
  • DevContainer configuration
  • Dependency update

AI Artifacts:

  • Reviewed contribution with prompt-builder agent and addressed all feedback
  • Copilot instructions (.github/instructions/*.instructions.md)
  • Copilot prompt (.github/prompts/*.prompt.md)
  • Copilot agent (.github/agents/*.agent.md)
  • Copilot skill (.github/skills/*/SKILL.md)

Other:

  • Script/automation (.ps1, .sh, .py)
  • Other (please describe):

Testing

  • Ran: npx markdownlint-cli2 "collections/*.collection.md" → 0 errors
  • Ran: npm run lint:md → 0 errors
  • Verified no regressions in markdown linting across repository

Checklist

Required Checks

  • Documentation is updated (if applicable)
  • Files follow existing naming conventions
  • Changes are backwards compatible (if applicable)
  • Tests added for new functionality (not applicable)

AI Artifact Contributions

  • Used /prompt-analyze to review contribution
  • Addressed all feedback from prompt-builder review
  • Verified contribution follows common standards and type-specific requirements

Required Automated Checks

  • Markdown linting: npm run lint:md
  • Spell checking: npm run spell-check
  • Frontmatter validation: npm run lint:frontmatter
  • Skill structure validation: npm run validate:skills
  • Link validation: npm run lint:md-links
  • PowerShell analysis: npm run lint:ps
  • Plugin freshness: npm run plugin:generate
  • Docusaurus tests: npm run docs:test

Security Considerations

  • This PR does not contain any sensitive or NDA information
  • Any new dependencies have been reviewed for security issues (none added)
  • Security-related scripts follow the principle of least privilege

@PratikWayase PratikWayase requested a review from a team as a code owner April 16, 2026 05:21
@katriendg
Copy link
Copy Markdown
Contributor

Thanks for your contribution @PratikWayase - I believe it will be good to hold off the changes on the collection MD files until PR #1316 is merged as it touches the same files, and you may have some conflicts. FYI @WilliamBerryiii

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.

Remove markdownlint exclusion for collections/*.collection.md

2 participants