compiler-research / CppInterOp

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

Add support for GetIncludePaths() #137

Closed Krishna-13-cyber closed 6 months ago

codecov[bot] commented 10 months ago

Codecov Report

Attention: 20 lines in your changes are missing coverage. Please review.

Comparison is base (530bb65) 63.66% compared to head (8264571) 63.36%. Report is 45 commits behind head on main.

:exclamation: Current head 8264571 differs from pull request most recent head 610d715. Consider uploading reports for the commit 610d715 to get more accurate results

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137/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/137?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 #137 +/- ## ========================================== - Coverage 63.66% 63.36% -0.31% ========================================== Files 16 16 Lines 4178 4198 +20 ========================================== Hits 2660 2660 - Misses 1518 1538 +20 ``` | [Files](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) | Coverage Δ | | |---|---|---| | [include/clang/Interpreter/CppInterOp.h](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-aW5jbHVkZS9jbGFuZy9JbnRlcnByZXRlci9DcHBJbnRlck9wLmg=) | `96.29% <ø> (ø)` | | | [lib/Interpreter/CppInterOp.cpp](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-bGliL0ludGVycHJldGVyL0NwcEludGVyT3AuY3Bw) | `73.46% <0.00%> (-0.10%)` | :arrow_down: | | [include/clang/Interpreter/CppInterOpInterpreter.h](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-aW5jbHVkZS9jbGFuZy9JbnRlcnByZXRlci9DcHBJbnRlck9wSW50ZXJwcmV0ZXIuaA==) | `55.93% <0.00%> (-6.34%)` | :arrow_down: | | [Files](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) | Coverage Δ | | |---|---|---| | [include/clang/Interpreter/CppInterOp.h](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-aW5jbHVkZS9jbGFuZy9JbnRlcnByZXRlci9DcHBJbnRlck9wLmg=) | `96.29% <ø> (ø)` | | | [lib/Interpreter/CppInterOp.cpp](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-bGliL0ludGVycHJldGVyL0NwcEludGVyT3AuY3Bw) | `73.46% <0.00%> (-0.10%)` | :arrow_down: | | [include/clang/Interpreter/CppInterOpInterpreter.h](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-aW5jbHVkZS9jbGFuZy9JbnRlcnByZXRlci9DcHBJbnRlck9wSW50ZXJwcmV0ZXIuaA==) | `55.93% <0.00%> (-6.34%)` | :arrow_down: |
vgvassilev commented 6 months ago

@Krishna-13-cyber can you rebase your PR?

Krishna-13-cyber commented 6 months ago

Hi @vgvassilev Yes I tried to rebase but there are set of conflicts, I will get this done shortly. Thanks.

Krishna-13-cyber commented 6 months ago

Closing this due to some excessive conflicts on my side.