Skip to content

Specify PROJECT_SOURCE_DIR instead of CMAKE_SOURCE_DIR#475

Merged
greenc-FNAL merged 1 commit intoFramework-R-D:mainfrom
knoepfel:version-header-location
Mar 30, 2026
Merged

Specify PROJECT_SOURCE_DIR instead of CMAKE_SOURCE_DIR#475
greenc-FNAL merged 1 commit intoFramework-R-D:mainfrom
knoepfel:version-header-location

Conversation

@knoepfel
Copy link
Copy Markdown
Member

This PR accommodates MPD projects, where PROJECT_SOURCE_DIR and CMAKE_SOURCE_DIR are not the same.

@knoepfel knoepfel requested a review from greenc-FNAL March 30, 2026 21:53
@greenc-FNAL greenc-FNAL merged commit 5ff4730 into Framework-R-D:main Mar 30, 2026
32 checks passed
@knoepfel knoepfel deleted the version-header-location branch March 30, 2026 22:01
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main     #475      +/-   ##
==========================================
- Coverage   85.43%   85.38%   -0.06%     
==========================================
  Files         144      144              
  Lines        3591     3591              
  Branches      615      615              
==========================================
- Hits         3068     3066       -2     
  Misses        315      315              
- Partials      208      210       +2     
Flag Coverage Δ
unittests 85.38% <ø> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7a2dab...f34d411. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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