Skip to content

fix: put buildings guide MDX imports on separate lines - #577

Merged
John McCall (lowlydba) merged 1 commit into
mainfrom
fix/llms-full-mdx-imports
Oct 14, 2026
Merged

John McCall (lowlydba) merged 1 commit into
mainfrom
fix/llms-full-mdx-imports

Conversation

@lowlydba

Copy link
Copy Markdown
Contributor

Pull Request

Splits the five imports on line 7 of docs/guides/buildings/index.mdx onto separate lines so the docusaurus-plugin-llms 0.6.1 excludeImports option strips them from llms-full.txt. Fixes #576

Testing

Verified locally: npm run build then npx vitest run passes, including the llms-full.txt import check that failed on main.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Signed-off-by: John McCall <john@overturemaps.org>
@github-actions

Copy link
Copy Markdown

🗺️ Overture Docs preview is live!

🌍 Preview site https://staging.overturemaps.org/docs/pr/577/index.html
🕐 Updated Oct 14, 2026 19:21 UTC
📝 Commit 14b63bc

Note

♻️ This preview updates automatically with each push to this PR.

@github-actions

Copy link
Copy Markdown

✅MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Max errors Warnings Elapsed time
✅ JAVASCRIPT eslint 33 0 0 22.24s
✅ MARKDOWN markdownlint 286 0 0 0.63s
✅ SQL sqlfluff 79 0 0 6.93s

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 FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@10.1.0 --custom-flavor-setup --custom-flavor-linters JAVASCRIPT_ES,MARKDOWN_MARKDOWNLINT,SQL_SQLFLUFF

MegaLinter is provided by OX Security
Show us your support by starring ⭐ the repository

@lowlydba John McCall (lowlydba) changed the title [FIX] Put buildings guide MDX imports on separate lines fix: put buildings guide MDX imports on separate lines Oct 14, 2026
@lowlydba
John McCall (lowlydba) marked this pull request as ready for review October 14, 2026 19:21
@lowlydba
John McCall (lowlydba) requested a review from a team as a code owner October 14, 2026 19:21
Copilot AI balanced review requested due to automatic review settings October 14, 2026 19:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 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.

@lowlydba
John McCall (lowlydba) merged commit f360917 into main Oct 14, 2026
19 of 20 checks passed
@lowlydba
John McCall (lowlydba) deleted the fix/llms-full-mdx-imports branch October 14, 2026 19:28

This branch was successfully deployed

1 active deployment
staging — 14b63bc3 Deployed Oct 14, 2026 by lowlydba via Deploy #583
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.

Production Deploy fails: llms-full.txt contains MDX import statements

3 participants