compiler-research / CppInterOp

A Clang-based C++ Interoperability Library
Other
38 stars 20 forks source link

Add xeus-cpp build to ci + remove xeus-clang-repl builds from ci #258

Closed mcbarton closed 2 months ago

mcbarton commented 2 months ago

@vgvassilev This PR adds back what is needed to build xeus-cpp as part of the ci. I consider this PR ready for merging, as it was tested previously. It just wasn't merged at the time as CppInterOp wasn't part of compiler-research/xeus-cpp .

Fixes https://github.com/compiler-research/CppInterOp/issues/257

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.24%. Comparing base (f93e7b6) to head (0f23735). Report is 12 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/258/graphs/tree.svg?width=650&height=150&src=pr&token=7UWTYSVVT5&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research)](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/258?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) ```diff @@ Coverage Diff @@ ## main #258 +/- ## ========================================== - Coverage 87.65% 79.24% -8.42% ========================================== Files 4 8 +4 Lines 1742 2997 +1255 ========================================== + Hits 1527 2375 +848 - Misses 215 622 +407 ``` [see 8 files with indirect coverage changes](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/258/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) [see 8 files with indirect coverage changes](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/258/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research)
vgvassilev commented 2 months ago

Yes, no point in having xeus-clang-repl as part of the builds. They already are bulky enough..

mcbarton commented 2 months ago

@vgvassilev I've added to this PR a commit that removes everything relating to xeus-clang-repl from the ci.