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

Small fix #154

Merged
merged 3 commits into from
Nov 10, 2023
Merged

Small fix #154

merged 3 commits into from
Nov 10, 2023

Conversation

misonijnik
Copy link
Collaborator

No description provided.

@misonijnik misonijnik force-pushed the misonijnik/fixes branch 2 times, most recently from 8007332 to 7e89b7e Compare November 10, 2023 01:30
@codecov-commenter
Copy link

Codecov Report

Merging #154 (6651e89) into main (ccf75c6) will increase coverage by 1.37%.
The diff coverage is 59.09%.

❗ Current head 6651e89 differs from pull request most recent head a1eac5c. Consider uploading reports for the commit a1eac5c to get more accurate results

@@            Coverage Diff             @@
##             main     #154      +/-   ##
==========================================
+ Coverage   64.38%   65.75%   +1.37%     
==========================================
  Files         215      221       +6     
  Lines       27693    29800    +2107     
  Branches     6177     6568     +391     
==========================================
+ Hits        17831    19596    +1765     
- Misses       7572     7708     +136     
- Partials     2290     2496     +206     
Files Coverage Δ
lib/Core/Executor.cpp 73.52% <100.00%> (-0.78%) ⬇️
lib/Expr/Constraints.cpp 84.00% <100.00%> (+0.33%) ⬆️
lib/Expr/IndependentSet.cpp 80.80% <55.00%> (-2.39%) ⬇️

... and 44 files with indirect coverage changes

@misonijnik misonijnik force-pushed the misonijnik/fixes branch 3 times, most recently from b2d6a4a to 733d4c8 Compare November 10, 2023 15:17
@misonijnik misonijnik merged commit 82b0cfd into main Nov 10, 2023
20 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