fix(tier-check): add missing markdown table headers for repository health#239
Open
anubhav756 wants to merge 1 commit intomodelcontextprotocol:mainfrom
Open
fix(tier-check): add missing markdown table headers for repository health#239anubhav756 wants to merge 1 commit intomodelcontextprotocol:mainfrom
anubhav756 wants to merge 1 commit intomodelcontextprotocol:mainfrom
Conversation
…alth Fixes a bug where the Markdown output format for the Repository Health section lacked a table header, causing it to render as raw text instead of a table on GitHub.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes a bug in the Markdown generator for the
tier-checktool where the Repository Health section was missing a table header.Motivation and Context
When running
npx @modelcontextprotocol/conformance tier-check --output markdown, the resulting Markdown for the "Repository Health" section lacked a header row and separator row. According to the GFM specification, tables without headers are not parsed as HTML tables. This caused the output to render as raw, unformatted text when piped into GitHub Issues. This change adds the missing| Metric | Status | Details |header to ensure proper rendering.How Has This Been Tested?
Tested locally by running the
tier-checkcommand with--output markdownagainst an SDK repository and verifying that the resulting output renders correctly as a 3-column table in a standard GitHub Issue markdown preview.Breaking Changes
None. Users consuming the Markdown output will simply see it render correctly now.
Types of changes
Checklist
Additional context
This is particularly useful for SDK maintainers who use GitHub Actions to pipe the output of the
tier-checkcommand directly into automated, nightly GitHub Issues to track their operational SLAs!