Skip to content

markdownlint: replace MD033 global disable with element allowlist#422

Merged
jbampton merged 2 commits intomainfrom
copilot/whitelist-specific-html-elements
Apr 4, 2026
Merged

markdownlint: replace MD033 global disable with element allowlist#422
jbampton merged 2 commits intomainfrom
copilot/whitelist-specific-html-elements

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 4, 2026

Globally disabling MD033 permits arbitrary inline HTML in Markdown, undermining formatting consistency. The rule should be scoped to only the elements the project actually uses.

Changes

  • .github/linters/.markdown-lint.yml: Replace MD033: false with an explicit allowlist containing only the HTML elements present in the repo's Markdown files (<div> and <h1>, both used in README.md for centered header layout)
# Before
MD033: false

# After
MD033:
  allowed_elements: ["div", "h1"]

Copilot AI changed the title [WIP] Update markdownlint to whitelist necessary HTML elements markdownlint: replace MD033 global disable with element allowlist Apr 4, 2026
Copilot AI requested a review from jbampton April 4, 2026 09:20
@jbampton jbampton added this to Next Apr 4, 2026
@github-project-automation github-project-automation bot moved this to In progress in Next Apr 4, 2026
@jbampton jbampton added this to the Hackfest milestone Apr 4, 2026
@jbampton jbampton marked this pull request as ready for review April 4, 2026 09:22
Copy link
Copy Markdown
Member

@jbampton jbampton left a comment

Choose a reason for hiding this comment

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

🏂

@jbampton jbampton merged commit 9b20e6b into main Apr 4, 2026
12 of 13 checks passed
@jbampton jbampton deleted the copilot/whitelist-specific-html-elements branch April 4, 2026 09:27
@github-project-automation github-project-automation bot moved this from In progress to Done in Next Apr 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

markdownlint: whitelist only the specific HTML elements that are necessary for your project

2 participants