Skip to content
This repository has been archived by the owner on Dec 15, 2023. It is now read-only.

Predeploy token fee and account contracts #115

Merged
merged 62 commits into from
Jun 3, 2022
Merged

Predeploy token fee and account contracts #115

merged 62 commits into from
Jun 3, 2022

Conversation

FabijanC
Copy link
Collaborator

@FabijanC FabijanC commented May 31, 2022 •
edited
Loading

Usage related changes

Development related changes

  • Introduce psutil dev dependency for postman testing (for killing of children processes)

Checklist:

  • Performed a self-review of the code
  • Rebased to the base branch
  • Documented the changes
  • Updated the tests
  • All tests are passing

FabijanC added 30 commits April 20, 2022 11:37
@FabijanC
Predeploy fee token [skip ci]
d4fd6b4
@FabijanC
Predeploy account [skip ci]
da569bb
@FabijanC
Add initial_balance to accounts (WIP)
da5c9c9
@FabijanC
Introduce predeployment CLI options (WIP) [skip ci]
ee33376
@FabijanC
Create StarknetContract instance (WIP) [skip ci]
6eba25c
@FabijanC
Merge branch 'master' into predeploy
c150ea6
@FabijanC
Merge branch 'master' into predeploy
138075d
@FabijanC
Fix contract definition and address for fee token
edce3ba
@FabijanC
Merge branch 'master' into predeploy
e0ab460
@FabijanC
Merge branch 'master' into predeploy
dfa2586
@FabijanC
Correctly handle uint256; set gas_price on block
aee8221
@FabijanC
Merge branch 'master' into predeploy
36a57c4
@FabijanC
Refactor wrapper initialization
01bb841
@FabijanC
Remove custom fee check
dd37930
@FabijanC
Support getting account balance
ba089ff
@FabijanC
Merge branch 'master' into predeploy
1229dcf
@FabijanC
Remove debug info from erc20 artifact
577dc0f
@FabijanC
Cleanup TODOs
d24af2c
@FabijanC
Allow int or float for gas_price and balance
97d183c
@FabijanC
Start testing
95d3dce
@FabijanC
Merge branch 'master' into predeploy
14f25b4
@FabijanC
Fix reset
36f90b0
@FabijanC
Merge branch 'master' into predeploy
bbd03cc
@FabijanC
Improve clean-up in dump tests
b9a4df1 
* Introduce psutil dev dependency
@FabijanC
Fix cleanup in postman test
6981557
@FabijanC
Fix initialization
9c34909
@FabijanC
Fix fee estimation
1fad647
@FabijanC
Issue a warning for a parsing imprecision
d6ceeb2
@FabijanC
Remove logging of token address
4b600f5
@FabijanC
Merge branch 'master' into predeploy
e6232f8
@FabijanC FabijanC marked this pull request as ready for review June 1, 2022 17:05
badurinantun
badurinantun suggested changes Jun 2, 2022
View reviewed changes
Copy link
Contributor

@badurinantun badurinantun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, no major requests

starknet_devnet/account.py Show resolved Hide resolved
starknet_devnet/account.py Show resolved Hide resolved
starknet_devnet/blueprints/base.py Outdated Show resolved Hide resolved
starknet_devnet/fee_token.py Show resolved Hide resolved
starknet_devnet/server.py Outdated Show resolved Hide resolved
starknet_devnet/state.py Show resolved Hide resolved
starknet_devnet/util.py Outdated Show resolved Hide resolved
starknet_devnet/util.py Show resolved Hide resolved
starknet_devnet/util.py Show resolved Hide resolved
starknet_devnet/util.py Outdated Show resolved Hide resolved
@FabijanC FabijanC merged commit a336e7c into master Jun 3, 2022
@FabijanC FabijanC deleted the predeploy branch June 3, 2022 09:02
@cptartur cptartur mentioned this pull request Jun 9, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@badurinantun badurinantun badurinantun approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

A list of pre-funded test accounts
2 participants
@FabijanC @badurinantun