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

Add pre-validation checks to PyValidator #1100

Conversation

giladchase
Copy link
Collaborator

@giladchase giladchase commented Nov 7, 2023

@giladchase giladchase changed the base branch from main to gilad/reconnect-native-blockifier November 7, 2023 21:38
@giladchase giladchase changed the title Gilad/pre validation in pre validator Add pre-validation checks to PyValidator Nov 7, 2023
@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2023

Codecov Report

Merging #1100 (3df0a3d) into gilad/reconnect-native-blockifier (f9174a9) will decrease coverage by 0.29%.
The diff coverage is 0.00%.

@@                          Coverage Diff                          @@
##           gilad/reconnect-native-blockifier    #1100      +/-   ##
=====================================================================
- Coverage                              67.58%   67.29%   -0.29%     
=====================================================================
  Files                                     51       51              
  Lines                                   6501     6529      +28     
  Branches                                6501     6529      +28     
=====================================================================
  Hits                                    4394     4394              
- Misses                                  1723     1751      +28     
  Partials                                 384      384              
Files Coverage Δ
crates/native_blockifier/src/py_validator.rs 0.00% <0.00%> (ø)

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@giladchase giladchase requested review from elintul and noaov1 November 7, 2023 21:55
@giladchase giladchase closed this Nov 8, 2023
gswirski pushed a commit to reilabs/blockifier that referenced this pull request Jun 26, 2024
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