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

chore: expose compilation args for the compile sierra to casm util #348

Conversation

ArniStarkware
Copy link
Contributor

@ArniStarkware ArniStarkware commented Jul 2, 2024

This change is Reviewable

Copy link
Contributor Author

ArniStarkware commented Jul 2, 2024

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

@ArniStarkware ArniStarkware marked this pull request as ready for review July 2, 2024 15:04
@codecov-commenter
Copy link

codecov-commenter commented Jul 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.46%. Comparing base (05697a3) to head (a59234b).

Additional details and impacted files
@@                                Coverage Diff                                @@
##           arni/declare/post_compilation/size_validation     #348      +/-   ##
=================================================================================
- Coverage                                          81.56%   81.46%   -0.10%     
=================================================================================
  Files                                                 42       42              
  Lines                                               1833     1818      -15     
  Branches                                            1833     1818      -15     
=================================================================================
- Hits                                                1495     1481      -14     
  Misses                                               265      265              
+ Partials                                              73       72       -1     

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

@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/compilation_file branch from 8169305 to 4733c8e Compare July 2, 2024 15:43
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 3e9c6fe to 4053ee7 Compare July 2, 2024 15:45
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/compilation_file branch from 4733c8e to 2546a4b Compare July 3, 2024 07:15
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 4053ee7 to 34edd3c Compare July 3, 2024 07:15
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/compilation_file branch from 2546a4b to 6a332d4 Compare July 3, 2024 09:12
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 34edd3c to 959f42e Compare July 3, 2024 09:12
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/compilation_file branch from 6a332d4 to 0f229c0 Compare July 3, 2024 10:03
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 959f42e to 8224c25 Compare July 3, 2024 10:04
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/compilation_file branch from 0f229c0 to 93e2b1b Compare July 3, 2024 12:02
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch 2 times, most recently from 23ee4bd to 59b2d0b Compare July 3, 2024 12:36
@ArniStarkware ArniStarkware changed the base branch from arni/declare/compilation/compilation_file to main July 3, 2024 12:36
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch 4 times, most recently from bb34d13 to 7771a0d Compare July 15, 2024 13:28
@ArniStarkware ArniStarkware changed the base branch from main to arni/declare/post_compilation/size_validation July 15, 2024 13:28
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 7771a0d to bd21d61 Compare July 15, 2024 13:35
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from caf5d12 to 2f6a155 Compare July 17, 2024 07:56
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 10697c5 to 825012e Compare July 18, 2024 12:48
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 2f6a155 to 13c0bc6 Compare July 18, 2024 12:48
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 825012e to 14ff21e Compare July 18, 2024 13:28
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 13c0bc6 to eec30dd Compare July 18, 2024 13:29
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 14ff21e to 1973ad9 Compare July 22, 2024 10:36
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from eec30dd to 35d7910 Compare July 22, 2024 10:36
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 1973ad9 to 8f817da Compare July 22, 2024 11:07
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 35d7910 to 4fe8181 Compare July 22, 2024 11:07
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 8f817da to 77d31d9 Compare July 22, 2024 11:14
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 4fe8181 to a033681 Compare July 22, 2024 11:14
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 77d31d9 to b34b37f Compare July 23, 2024 10:47
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from a033681 to 0e7c3fa Compare July 23, 2024 10:47
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from b34b37f to 9a5fb02 Compare July 23, 2024 11:07
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 0e7c3fa to 857cad9 Compare July 23, 2024 11:07
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 9a5fb02 to 3ad7790 Compare July 23, 2024 11:10
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 857cad9 to 5ab9f5e Compare July 23, 2024 11:10
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 3ad7790 to 2a2be69 Compare July 23, 2024 13:55
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from 5ab9f5e to c36865c Compare July 23, 2024 13:55
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/size_validation branch from 2a2be69 to 05697a3 Compare July 23, 2024 14:16
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/expose_compilation_args branch from c36865c to a59234b Compare July 23, 2024 14:17
Copy link
Contributor

@yair-starkware yair-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 2 of 3 files at r2, 1 of 1 files at r3.
Reviewable status: 1 of 8 files reviewed, all discussions resolved (waiting on @dafnamatsry and @Yael-Starkware)

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