Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add helpers header for working with CPython in C++ tests #526

Merged

Conversation

jiridanek
Copy link
Contributor

@jiridanek jiridanek commented May 28, 2022

@jiridanek jiridanek added this to the 2.1.0 milestone May 28, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #526 (c3a9786) into main (6aaadd5) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #526      +/-   ##
==========================================
- Coverage   25.17%   25.17%   -0.01%     
==========================================
  Files         115      115              
  Lines       28771    28771              
  Branches     4669     4669              
==========================================
- Hits         7243     7242       -1     
  Misses      20561    20561              
- Partials      967      968       +1     
Flag Coverage Δ
unittests 25.17% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/router_core/address_watch.c 78.48% <0.00%> (-1.27%) ⬇️

Continue to review full report at Codecov.

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

@jiridanek jiridanek force-pushed the jd_2022_05_26_python_utils_hpp branch from c3a9786 to 9f9008e Compare June 1, 2022 21:40
@jiridanek jiridanek merged commit 8268639 into skupperproject:main Jun 3, 2022
@jiridanek jiridanek deleted the jd_2022_05_26_python_utils_hpp branch June 3, 2022 19:20
@jiridanek jiridanek self-assigned this Jun 16, 2022
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