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

Raise an error when deploy account does not run in validate mode. #1147

Closed

Conversation

ArniStarkware
Copy link
Contributor

@ArniStarkware ArniStarkware commented Nov 19, 2023

This change is Reviewable

@ArniStarkware ArniStarkware requested a review from noaov1 November 19, 2023 08:21
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (27a49ec) 67.15% compared to head (c2a9be6) 67.20%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1147      +/-   ##
==========================================
+ Coverage   67.15%   67.20%   +0.04%     
==========================================
  Files          52       52              
  Lines        6729     6739      +10     
  Branches     6729     6739      +10     
==========================================
+ Hits         4519     4529      +10     
+ Misses       1798     1797       -1     
- Partials      412      413       +1     

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

@ArniStarkware
Copy link
Contributor Author

This PR is replaced by: #1156.

@ArniStarkware ArniStarkware deleted the arni/starknet/execution_mode/deploy_account branch February 15, 2024 09:24
gswirski pushed a commit to reilabs/blockifier that referenced this pull request Jun 26, 2024
* core: upgradeable component wip

* #[dojo::contract] merge Event, update upgradeable

* update base_test.cairo & dojo-lang plugin_test_data

* add Upgraded event in upgradeable component

* move erc20, fix build issue, delete old tests

* update erc20 to use #[dojo::contract]

* revert changes on erc20

* revert changes on erc20

* fully qualify component

* handle when Event is not defined in contract

* follow clippy recomandation

* torii: update records_contract address

* trigger CI

* trigger CI

* rebase main

* update manifest_test_data

* use right records_contract address
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