fix(html/astro): keep a blank after the frontmatter#8039
Conversation
🦋 Changeset detectedLatest commit: e660ad8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
WalkthroughAdded a changelog file and updated tests and formatter so Astro frontmatter is followed by a blank line. The Astro test fixture was extended with extra blank lines and an HTML snippet. In the HTML root formatter, Suggested reviewers
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (11)📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-10-15T09:22:46.002ZApplied to files:
📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-10-15T09:22:46.002ZApplied to files:
📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-10-15T09:22:15.851ZApplied to files:
📚 Learning: 2025-09-13T16:16:06.459ZApplied to files:
🧬 Code graph analysis (1)crates/biome_html_formatter/src/html/auxiliary/root.rs (2)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (9)
🔇 Additional comments (1)
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. Comment |
7fcee59 to
48f26ca
Compare
Summary
Biome now keeps a blank line after the frontmatter section in Astro files.
Fix #8038
Test Plan
I added some blank lines to the
full_supporttest case to ensure that multiple blank lines are formatted into a single one.Note that some snapshot files were removed — I guess this means some leftover differences with Prettier have been eliminated. 🎉
Docs