Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: commit 'mdsnippets' changes #1310

Closed

Conversation

hsnabszhdn
Copy link

@hsnabszhdn hsnabszhdn commented Mar 15, 2025

I intended to contribute to docs, but even before I do anything, I ended up with 66 changes by running npm run docs:

2025-03-16_08-29

With this many changes, it's really hard to focus on the actual change I want to make/suggest.

It could be that mdsnippets has released a new version that is case sensitive now 🤷

$ mdsnippets --version
mdsnippets 27.0.2+1f22752e2764bfbe50a24b4bb424a91bf6dc82c9
Finished 44ms

Note

It would've been better if the version of mdsnippets was mentioned in README.md instead of implying the "latest" version. Otherwise, even if this PR gets merged, someone else with an older version of mdsnippets installed would raise another PR reverting all this.

@jeremydmiller
Copy link
Member

I'll do this independently against the master branch in a bit. Long story, but I don't like to have the mdsnippets changes committed because that's caused loss of content in the past on other projects.

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.

2 participants