Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

chore: verify Cairo0 compiler dependencies before compiling #1873

Open
wants to merge 1 commit into
base: dori/fetch-compiler-version
Choose a base branch
from

Conversation

dorimedini-starkware
Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware commented May 6, 2024

This change is Reviewable

@dorimedini-starkware dorimedini-starkware self-assigned this May 6, 2024
@codecov-commenter
Copy link

codecov-commenter commented May 6, 2024

Codecov Report

Attention: Patch coverage is 0% with 22 lines in your changes are missing coverage. Please review.

Project coverage is 77.05%. Comparing base (aed63a4) to head (51a949c).

Files Patch % Lines
crates/blockifier/src/test_utils/cairo_compile.rs 0.00% 22 Missing ⚠️
Additional details and impacted files
@@                       Coverage Diff                       @@
##           dori/fetch-compiler-version    #1873      +/-   ##
===============================================================
- Coverage                        77.24%   77.05%   -0.20%     
===============================================================
  Files                               61       61              
  Lines                             8671     8693      +22     
  Branches                          8671     8693      +22     
===============================================================
  Hits                              6698     6698              
- Misses                            1534     1556      +22     
  Partials                           439      439              

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

@dorimedini-starkware dorimedini-starkware force-pushed the dori/verify-cairo0-compiler-deps branch from 9d2d44c to ffdd1b0 Compare May 7, 2024 09:29
@dorimedini-starkware dorimedini-starkware force-pushed the dori/fetch-compiler-version branch from 76dcca0 to aed63a4 Compare May 7, 2024 10:17
Signed-off-by: Dori Medini <dori@starkware.co>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants