Skip to content

Conversation

@mizvekov
Copy link
Collaborator

@mizvekov mizvekov commented Oct 20, 2025

This change makes mrdocs use the system includes by default instead of the bundled ones.

This helps mrdocs build projects which rely on differences between standard library implementations, since now mrdocs sees the source code closer to what the original compiler did, with only potential differences in the compilers themselves and some minor command line manipulations remaining.

It also stops bundling the clang resource directory, relying on the one installed with libclang.
This makes it so installing both to the same prefix be necessary, which is the same as all the other libclang based tools.

@mizvekov mizvekov self-assigned this Oct 20, 2025
@mizvekov mizvekov force-pushed the default_use_system_libs branch from 45090b7 to 0bd333b Compare October 20, 2025 20:34
@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

❌ Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.41%. Comparing base (c136a46) to head (11a7b1e).

Files with missing lines Patch % Lines
src/lib/MrDocsSettingsDB.cpp 0.00% 2 Missing ⚠️

❌ Your patch check has failed because the patch coverage (83.33%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1077      +/-   ##
===========================================
+ Coverage    86.19%   86.41%   +0.22%     
===========================================
  Files          323      323              
  Lines        23954    23955       +1     
===========================================
+ Hits         20646    20700      +54     
+ Misses        3308     3255      -53     

☔ View full report in Codecov by Sentry.
📢 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.

@mizvekov mizvekov force-pushed the default_use_system_libs branch from 0bd333b to 53cefa2 Compare October 21, 2025 16:31
@mizvekov mizvekov force-pushed the default_use_system_libs branch 3 times, most recently from 8e86b01 to e7ae03a Compare November 24, 2025 18:55
@mizvekov mizvekov force-pushed the default_use_system_libs branch 4 times, most recently from f739971 to 244d6ce Compare November 26, 2025 19:31
@mizvekov mizvekov force-pushed the default_use_system_libs branch 5 times, most recently from 44a71c2 to 350cc17 Compare December 3, 2025 18:45
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

🚧 Danger.js checks for MrDocs are experimental; expect some rough edges while we tune the rules.

⚠️ Warnings

Warning

PR description looks empty. Please add a short rationale and testing notes.

✨ Highlights

  • 🧪 Existing golden tests changed (behavior likely shifted)

🧾 Changes by Scope

Scope Lines Δ Lines + Lines - Files Δ Files + Files ~ Files ↔ Files -
⚙️ CI 61 45 16 1 - 1 - -
🥇 Golden Tests 49 27 22 9 4 5 - -
🧪 Unit Tests 46 41 5 4 1 3 - -
🛠️ Source 37 23 14 6 - 6 - -
📄 Docs 31 25 6 3 - 3 - -
🏗️ Build / Toolchain 28 12 16 2 - 2 - -
🧰 Tooling 6 5 1 2 - 2 - -
Total 258 178 80 27 5 22 - -

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

🔝 Top Files

  • .github/workflows/ci.yml (CI): 61 lines Δ (+45 / -16)
  • test-files/golden-tests/symbols/function/sfinae.xml (Golden Tests): 34 lines Δ (+17 / -17)
  • test-files/include/type_traits (Unit Tests): 32 lines Δ (+32 / -0)

Generated by 🚫 dangerJS against 11a7b1e

@mizvekov mizvekov force-pushed the default_use_system_libs branch 12 times, most recently from bb8d10e to be98305 Compare December 5, 2025 20:40
@mizvekov mizvekov force-pushed the default_use_system_libs branch 8 times, most recently from e86cab5 to 32ca7a3 Compare December 12, 2025 19:15
@mizvekov mizvekov force-pushed the default_use_system_libs branch 7 times, most recently from 5023cfd to 565c485 Compare December 17, 2025 17:31
@mizvekov mizvekov force-pushed the default_use_system_libs branch 6 times, most recently from e1ead36 to 3bf984d Compare December 24, 2025 04:17
This change makes mrdocs use the system includes by default instead of
the bundled ones.

This helps mrdocs build projects which rely on differences between
standard library implementations, since now mrdocs sees the source code
closer to what the original compiler did, with only potential differences in the
compilers themselves and some minor command line manipulations remaining.

It also stops bundling the clang resource directory, relying on the one
installed with libclang. This makes it so installing both to the same
prefix be necessary, which is the same as all the other libclang based tools.
@mizvekov mizvekov force-pushed the default_use_system_libs branch from 3bf984d to 11a7b1e Compare December 29, 2025 15:01
@mizvekov mizvekov marked this pull request as ready for review December 29, 2025 15:05
@cppalliance-bot
Copy link

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

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

2025-12-29 15:12:36 UTC

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.

3 participants