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: use builtins iter to define os_order_iter #351

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

@codecov-commenter
Copy link

codecov-commenter commented Jul 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.70%. Comparing base (dba37cb) to head (56c0a93).

Additional details and impacted files
@@                                         Coverage Diff                                         @@
##           arni/declare/post_compilation/validation/trait_for_builtin_name     #351      +/-   ##
===================================================================================================
+ Coverage                                                            82.58%   82.70%   +0.11%     
===================================================================================================
  Files                                                                   33       33              
  Lines                                                                 1631     1642      +11     
  Branches                                                              1631     1642      +11     
===================================================================================================
+ Hits                                                                  1347     1358      +11     
  Misses                                                                 219      219              
  Partials                                                                65       65              

☔ 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 4733c8e to 2546a4b Compare July 3, 2024 07:15
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/builtins/os_order_iter branch from 63717fb to 4a3413b 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/builtins/os_order_iter branch 2 times, most recently from 1a35bd7 to 193d382 Compare July 3, 2024 09:47
@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/builtins/os_order_iter branch from 193d382 to 24a6ce6 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/builtins/os_order_iter branch from 24a6ce6 to 1c0bd82 Compare July 3, 2024 12:02
@ArniStarkware ArniStarkware changed the base branch from arni/declare/compilation/compilation_file to arni/declare/post_compilation/validation/trait_for_builtin_name July 3, 2024 12:02
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/validation/trait_for_builtin_name branch from 69054eb to 65d2d27 Compare July 3, 2024 12:34
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/builtins/os_order_iter branch from 1c0bd82 to 33bb4e3 Compare July 3, 2024 12:34
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/validation/trait_for_builtin_name branch from 65d2d27 to cb35c83 Compare July 4, 2024 06:24
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/builtins/os_order_iter branch from 33bb4e3 to b47b3eb Compare July 4, 2024 06:24
@ArniStarkware ArniStarkware force-pushed the arni/declare/post_compilation/validation/trait_for_builtin_name branch from cb35c83 to dba37cb Compare July 4, 2024 06:25
@ArniStarkware ArniStarkware force-pushed the arni/declare/compilation/builtins/os_order_iter branch from b47b3eb to 56c0a93 Compare July 4, 2024 06:25
@ArniStarkware ArniStarkware deleted the arni/declare/compilation/builtins/os_order_iter branch August 21, 2024 07:27
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