Skip to content

Conversation

@MegaRedHand
Copy link
Contributor

@MegaRedHand MegaRedHand commented Apr 14, 2023

Description

Depends: #986

This PR adds the missing hint NewHint#28, now known as VERIFY_ZERO_V2. Also rename VERIFY_ZERO to VERIFY_ZERO_V1

@codecov
Copy link

codecov bot commented Apr 14, 2023

Codecov Report

Merging #989 (cbbf46d) into main (1a89550) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #989   +/-   ##
=======================================
  Coverage   97.76%   97.76%           
=======================================
  Files          75       75           
  Lines       31142    31155   +13     
=======================================
+ Hits        30446    30459   +13     
  Misses        696      696           
Impacted Files Coverage Δ
...int_processor/builtin_hint_processor_definition.rs 98.75% <100.00%> (+<0.01%) ⬆️
...ocessor/builtin_hint_processor/secp/field_utils.rs 98.74% <100.00%> (+0.04%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pefontana pefontana added the whitelisted-hint Implementation of hint on whitelist directory label Apr 14, 2023
Base automatically changed from newhint29-compute-doubling-slope to main April 15, 2023 03:58
@Oppen Oppen enabled auto-merge April 15, 2023 04:06
@Oppen Oppen added this pull request to the merge queue Apr 15, 2023
Merged via the queue into main with commit ce0d096 Apr 15, 2023
@Oppen Oppen deleted the newhint28-verify-zero branch April 15, 2023 04:49
kariy pushed a commit to dojoengine/cairo-rs that referenced this pull request Jun 23, 2023
* Add `COMPUTE_SLOPE_WHITELIST` hint

* Update changelog

* Fix: add missing return to cairo program

* Add `EC_DOUBLE_SCOPE_WHITELIST` hint

* Update changelog

* Add NewHint#28

* Update changelog

* Use name without WHITELIST

* Remove comments and add source

* Readd deleted comments

* Change assert_matches for assert + is_ok

---------

Co-authored-by: Mario Rugiero <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

whitelisted-hint Implementation of hint on whitelist directory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants