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 CLAP #147

Merged
merged 2 commits into from
Jun 22, 2022
Merged

add CLAP #147

merged 2 commits into from
Jun 22, 2022

Conversation

tdelabro
Copy link
Contributor

Add clap to executable

Description

Introduce better user interface for executable

@codecov-commenter
Copy link

codecov-commenter commented Jun 18, 2022

Codecov Report

Merging #147 (17538ea) into main (746d90e) will increase coverage by 0.04%.
The diff coverage is 76.92%.

@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
+ Coverage   96.29%   96.33%   +0.04%     
==========================================
  Files          24       24              
  Lines        4509     4507       -2     
==========================================
  Hits         4342     4342              
+ Misses        167      165       -2     
Impacted Files Coverage Δ
src/main.rs 0.00% <0.00%> (ø)
src/cairo_run.rs 100.00% <100.00%> (ø)
src/serde/deserialize_program.rs 92.79% <100.00%> (ø)
src/types/program.rs 100.00% <100.00%> (ø)
tests/bitwise_test.rs 100.00% <100.00%> (ø)
tests/cairo_run_test.rs 100.00% <100.00%> (ø)
tests/pedersen_test.rs 100.00% <100.00%> (ø)
tests/struct_test.rs 100.00% <100.00%> (ø)

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 746d90e...17538ea. Read the comment docs.

@unbalancedparentheses
Copy link
Member

Thanks @tdelabro. Gonna test it in a moment!

@unbalancedparentheses
Copy link
Member

@pefontana @fmoletta @Juan-M-V check this and let's merge it. from what I have read and tested it seems to be fine.

@unbalancedparentheses unbalancedparentheses merged commit d6deba4 into lambdaclass:main Jun 22, 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.

3 participants