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

feat: add compile sierra to casm util #493

Conversation

ArniStarkware
Copy link
Contributor

@ArniStarkware ArniStarkware commented Jul 17, 2024

See many important comments on #24.
Also - there were interesting discussions on #35.


This change is Reviewable

@ArniStarkware ArniStarkware force-pushed the arni/sierra_to_casm/use_executable/try_5 branch from 8361d0c to 9f7e1d5 Compare July 17, 2024 11:21
@ArniStarkware ArniStarkware changed the base branch from main to arni/chore/starknet_sierra_compile/fix_get_abs_path July 17, 2024 11:21
Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on Graphite Graphite

@codecov-commenter
Copy link

codecov-commenter commented Jul 17, 2024

Codecov Report

Attention: Patch coverage is 94.73684% with 2 lines in your changes missing coverage. Please review.

Project coverage is 83.58%. Comparing base (77398c0) to head (7d36848).

Files Patch % Lines
crates/starknet_sierra_compile/src/compile.rs 94.59% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@                                   Coverage Diff                                   @@
##           arni/chore/starknet_sierra_compile/fix_get_abs_path     #493      +/-   ##
=======================================================================================
+ Coverage                                                83.38%   83.58%   +0.19%     
=======================================================================================
  Files                                                       37       37              
  Lines                                                     1782     1803      +21     
  Branches                                                  1782     1803      +21     
=======================================================================================
+ Hits                                                      1486     1507      +21     
  Misses                                                     218      218              
  Partials                                                    78       78              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ArniStarkware
Copy link
Contributor Author

The tests tend to fail sporadically. Need to check.

@ArniStarkware ArniStarkware force-pushed the arni/chore/starknet_sierra_compile/fix_get_abs_path branch from b00b20b to 77398c0 Compare July 22, 2024 14:15
@ArniStarkware ArniStarkware force-pushed the arni/sierra_to_casm/use_executable/try_5 branch from 9f7e1d5 to 7d36848 Compare July 22, 2024 14:16
@ArniStarkware ArniStarkware force-pushed the arni/chore/starknet_sierra_compile/fix_get_abs_path branch from 77398c0 to 71013e9 Compare July 23, 2024 14:03
@ArniStarkware ArniStarkware force-pushed the arni/sierra_to_casm/use_executable/try_5 branch from 7d36848 to 010f9fb Compare July 23, 2024 14:04
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