Skip to content

Update the project infrastructure using beman.infra#150

Merged
steve-downey merged 21 commits intobemanproject:mainfrom
steve-downey:infra
Dec 21, 2025
Merged

Update the project infrastructure using beman.infra#150
steve-downey merged 21 commits intobemanproject:mainfrom
steve-downey:infra

Conversation

@steve-downey
Copy link
Copy Markdown
Member

Import the beman infrastructure into optional, and update the presets and CI to current Examplar versions.
Fix, workaround, or exclude MSVC issues.
Exclude C++17 and old compilers.

Make sure the compilation database is correct and includes the std flag.
Use string manip to normalize the relative build path.
git-subtree-dir: infra
git-subtree-split: 68ad626
Import the beman infra project as a subtree and update.
Use beman-install-library from beman/infra
Use beman-install-library from beman/infra
Copy the workflows from exemplar, before replacing the ci workflow.
Add the cmake prefix path to old CI config.
Clang 17 is too old. C++17 is not supported.
Exclude the example for optional not being a range.
Clang 21 and libc++ with C++26 don't leak <vector>. Include it, as it should
be anyway.
MSVC is unhappy with the definition of free swap.
MSVC fails to convert some of the highly constrained test types.
call to immediate function is not a constant expression
MSVC treats the ternary as creating a temporary and value returns garbage for
rvalue ref optionals.
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 20, 2025

Pull Request Test Coverage Report for Build 20401568859

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.1%) to 99.038%

Files with Coverage Reduction New Missed Lines %
include/beman/optional/optional.hpp 2 98.99%
Totals Coverage Status
Change from base Build 20394734712: -0.1%
Covered Lines: 309
Relevant Lines: 312

💛 - Coveralls

@steve-downey steve-downey merged commit 8420b95 into bemanproject:main Dec 21, 2025
79 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