Repository navigation
fix: put buildings guide MDX imports on separate lines - #577
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Signed-off-by: John McCall <john@overturemaps.org>
🗺️ Overture Docs preview is live!
Note ♻️ This preview updates automatically with each push to this PR. |
✅MegaLinter analysis: Success
See detailed reports in MegaLinter artifacts Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining
|
There was a problem hiding this comment.
🟢 Approval recommended
The narrow formatting change preserves guide behavior and has no identified issues.
0 open findings
What changed in this PR
Adjusts the buildings guide’s import formatting so the LLM documentation export can remove imports from llms-full.txt.
Changes:
- Places five imports on separate lines without changing their semantics.
| File | Description |
|---|---|
| docs/guides/buildings/index.mdx | Separates imports for compatibility with excludeImports. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Pull Request
Splits the five imports on line 7 of
docs/guides/buildings/index.mdxonto separate lines so thedocusaurus-plugin-llms0.6.1excludeImportsoption strips them fromllms-full.txt. Fixes #576Testing
Verified locally:
npm run buildthennpx vitest runpasses, including thellms-full.txtimport check that failed onmain.