Skip to content

Reduce build requirements, and ensure they are enforced in testing#132

Merged
mborland merged 1 commit into
developfrom
remove_reqs
Jun 16, 2025
Merged

Reduce build requirements, and ensure they are enforced in testing#132
mborland merged 1 commit into
developfrom
remove_reqs

Conversation

@mborland

Copy link
Copy Markdown
Member

Closes: #131

@codecov

codecov Bot commented Jun 16, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.83%. Comparing base (7363de5) to head (ee39122).
Report is 2 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #132   +/-   ##
========================================
  Coverage    95.83%   95.83%           
========================================
  Files          114      114           
  Lines         8115     8115           
========================================
  Hits          7777     7777           
  Misses         338      338           

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 7363de5...ee39122. Read the comment docs.

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

@mborland
mborland merged commit 59a5eb8 into develop Jun 16, 2025
41 checks passed
@mborland
mborland deleted the remove_reqs branch June 16, 2025 19:19
@Lastique

Copy link
Copy Markdown
Member

Note that you should normally not need to add any requirements to tests (or any other downstream users of the library), beyond those which are required by the tests themselves. For that, you would add the requirements that need to be fulfilled by the library users in the usage-requirements of the library. That is, requirements is what library itself needs for compiling, and usage-requirements is what users need to fulfill in order to consume the library. For header-only libraries, those are the same.

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.

Remove unnecessary C++11 requirements

2 participants