Skip to content

Symlink-tolerant path filtering and related Boost.Capy fixes#1228

Merged
alandefreitas merged 5 commits into
cppalliance:developfrom
alandefreitas:develop
Jun 11, 2026
Merged

Symlink-tolerant path filtering and related Boost.Capy fixes#1228
alandefreitas merged 5 commits into
cppalliance:developfrom
alandefreitas:develop

Conversation

@alandefreitas

Copy link
Copy Markdown
Collaborator

These are a few small, urgent fixes we needed for the Boost.Capy reference docs. The changes are small and simple. The main one is the exception: the fix is short, but the bug behind it was subtle, and figuring out what was going wrong took far longer than fixing it.

The compilation database was describing each file under the symlinked path, while MrDocs resolved paths to their real location. The path filters compared those two spellings literally, so nothing matched and the corpus was empty. The new policy is that inclusion is now generous (resolved path or as spelled) and exclusion stays strict (written path only).

Changes

Testing

Unit tests for the new path functions. The other fixes are covered by the existing golden suites.

Documentation

The configuration help for the input and exclude filters now explicitly explains the policy.

The error-on-empty-corpus option compared the corpus against empty(), but the global namespace is always extracted and protected from pruning, so the corpus is never truly empty after a successful build, and the option never triggered.
@github-actions

Copy link
Copy Markdown

✨ Highlights

  • 🧪 Existing golden tests changed (behavior likely shifted)

🧾 Changes by Scope

Scope Lines Δ% Lines Δ Lines + Lines - Files Δ Files + Files ~ Files ↔ Files -
🔧 Toolchain 45% 1411 1356 55 11 2 9 - -
🔧 Toolchain Tests 25% 766 749 17 5 1 4 - -
🛠️ Source 17% 527 264 263 12 - 11 - 1
🥇 Golden Tests 9% 282 254 28 36 - 36 - -
🧪 Unit Tests 4% 121 106 15 1 - 1 - -
📄 Docs <1% 6 3 3 1 - 1 - -
⚙️ CI <1% 1 1 - 1 - 1 - -
Total 100% 3114 2733 381 67 3 63 - 1

Legend: Files + (added), Files ~ (modified), Files ↔ (renamed), Files - (removed)

🔝 Top Files

  • util/bootstrap/src/configs/justfile.py (Toolchain): 792 lines Δ (+792 / -0)
  • util/bootstrap/tests/test_justfile.py (Toolchain Tests): 496 lines Δ (+496 / -0)
  • util/bootstrap/src/configs/run_configs.json (Toolchain): 293 lines Δ (+293 / -0)

Generated by 🚫 dangerJS against 02aea57

@cppalliance cppalliance deleted a comment from chatgpt-codex-connector Bot Jun 11, 2026
@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.17234% with 69 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.16%. Comparing base (b28ba4b) to head (02aea57).

Files with missing lines Patch % Lines
util/bootstrap/src/configs/justfile.py 86.64% 35 Missing and 18 partials ⚠️
util/bootstrap/src/configs/run_configs.py 80.00% 6 Missing and 2 partials ⚠️
util/bootstrap/src/installer.py 76.00% 4 Missing and 2 partials ⚠️
util/bootstrap/src/configs/clion.py 88.23% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1228      +/-   ##
===========================================
+ Coverage    81.97%   83.16%   +1.18%     
===========================================
  Files           34       35       +1     
  Lines         3179     3658     +479     
  Branches       743      843     +100     
===========================================
+ Hits          2606     3042     +436     
- Misses         392      409      +17     
- Partials       181      207      +26     
Flag Coverage Δ
bootstrap 83.16% <86.17%> (+1.18%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cppalliance-bot

Copy link
Copy Markdown

An automated preview of the documentation is available at https://1228.mrdocs.prtest2.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-06-11 05:50:20 UTC

@alandefreitas alandefreitas merged commit 3d254b6 into cppalliance:develop Jun 11, 2026
48 of 50 checks passed
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