fix: add --no-error-on-unmatched-pattern to oxfmt md rule in lint-staged

[ltyu]

Summary

• Added --no-error-on-unmatched-pattern to the *.md oxfmt rule in .lintstagedrc. oxfmt supports markdown but reads .prettierignore by default, which ignores .changeset/. When the only staged .md file is in an ignored directory, oxfmt exits with "Expected at least one target file", failing the pre-commit hook.

--no-error-on-unmatched-pattern — Do not exit with error when pattern is unmatched                                                                                                  
                                                                                                                                                                                      
  Normally when oxfmt receives a file path but skips it (e.g. because .prettierignore excludes it), it exits with code 2 and "Expected at least one target file." This flag makes it  
  exit 0 instead — a silent no-op when there's nothing to format.  

Drive-by changes

None

Related issues

Introduced in #7973 (prettier-to-oxfmt migration)

Backward compatibility

Yes

Testing

Verified committing .changeset/*.md files no longer fails the pre-commit hook.

[coderabbitai]

📝 Walkthrough

Walkthrough

The lint-staged configuration has been simplified by removing the Markdown formatting enforcement rule. The *.md pattern and its associated oxfmt --write command have been deleted, so Markdown files will no longer be automatically formatted through the lint-staged pipeline.

Changes

Cohort / File(s)	Summary
Configuration Update .lintstagedrc	Removed Markdown file formatting rule (*.md: oxfmt --write). Markdown files are no longer processed by lint-staged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

Markdown's no longer bound by the linter's rule,
Like an ogre's swamp, sometimes simpler is cool,
No more layers of formatting, one less thing to do,
The config just got cleaner—and that's just true! 🟢

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The PR title claims to add a flag, but the actual change removes the markdown entry entirely from the configuration.	Update the title to accurately reflect the change: 'fix: remove unsupported *.md entry from oxfmt lint-staged config' (which matches the PR objectives summary).
Description check	❓ Inconclusive	PR description addresses the core issue but doesn't fully match the template structure with clear sections for all required information.	Restructure the description to explicitly follow the template with clear ### Description, ### Drive-by changes, ### Related issues, ### Backward compatibility, and ### Testing sections as specified.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch le/fix-oxfmt-lintstagedrc

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}