Lowercase all mdsnippets snippet names#2534
Merged
jeremydmiller merged 1 commit intomainfrom Apr 17, 2026
Merged
Conversation
The latest MarkdownSnippets.Tool (28.2.0) is case-SENSITIVE when matching <!-- snippet: NAME --> references against #region NAME declarations. Older versions (e.g. 24.5.1) were case-insensitive, but contributors installing the current release would get build failures on any mixed-case mismatch. This commit lowercases every snippet name across the repo: - All <!-- snippet: NAME --> references in *.md files - All #region NAME declarations whose name uses the snippet naming convention (sample_* / snippet_* prefix) Code-organization regions with multi-word names like "#region Designer generated code" are left untouched. Verified by running mdsnippets 28.2.0 (the latest version) against the entire repo with no missing-snippet failures. The existing v24.5.1 install also continues to work. 500 files changed (mostly markdown — mdsnippets regenerates the inner anchor IDs, which now use the lowercased names). C# changes are preprocessor-directive renames only — no behavioral impact. Closes #2530 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Apr 21, 2026
Closed
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.
Summary
Closes #2530.
Why
The latest
MarkdownSnippets.Tool(28.2.0) is case-sensitive when matching<!-- snippet: NAME -->references against#region NAMEdeclarations. Older versions (e.g. 24.5.1) were case-insensitive — but contributors installing the current release would hit build failures on any mixed-case mismatch.Verified empirically with both versions:
<!-- snippet: SAMPLE_X -->against#region sample_x→ matches ✓Failed: Missing snippets: SAMPLE_X✗This PR forces every snippet name to lowercase so the docs build cleanly on any version.
What changed
<!-- snippet: NAME -->references in*.mdlowercased#region NAMEdeclarations that follow snippet naming conventions (sample_*/snippet_*prefix) lowercased#region Designer generated code,#region Test setup) left untouched499 files changed (mostly markdown).
Verification
Clean run, no missing-snippet failures. Also verified the existing 24.5.1 install still works.
🤖 Generated with Claude Code