Skip to content

Change test module makefile to cmake #623

New issue

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

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

Already on GitHub? # to your account

Merged
merged 2 commits into from
Mar 2, 2021

Conversation

alonre24
Copy link
Collaborator

@alonre24 alonre24 commented Mar 2, 2021

No description provided.

@alonre24 alonre24 requested a review from rafie March 2, 2021 13:48
@alonre24 alonre24 self-assigned this Mar 2, 2021
@codecov
Copy link

codecov bot commented Mar 2, 2021

Codecov Report

Merging #623 (30c2d48) into master (147ca89) will increase coverage by 1.81%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #623      +/-   ##
==========================================
+ Coverage   74.51%   76.32%   +1.81%     
==========================================
  Files          39       41       +2     
  Lines        6054     6517     +463     
==========================================
+ Hits         4511     4974     +463     
  Misses       1543     1543              
Impacted Files Coverage Δ
src/background_workers.c 79.16% <0.00%> (-1.39%) ⬇️
tests/module/LLAPI.c 74.72% <0.00%> (ø)
tests/module/DAG_utils.c 88.96% <0.00%> (ø)
src/model.c 78.28% <0.00%> (+1.14%) ⬆️
src/redisai.h 96.25% <0.00%> (+96.25%) ⬆️

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 147ca89...30c2d48. Read the comment docs.

@alonre24 alonre24 merged commit 19f7305 into master Mar 2, 2021
@alonre24 alonre24 deleted the Change_test_module_makefile_to_cmake branch March 2, 2021 15:40
alonre24 added a commit that referenced this pull request Mar 16, 2021
…make

Change test module makefile to cmake
@chayim chayim mentioned this pull request Jul 22, 2021
@chayim chayim mentioned this pull request Nov 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants