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

0.11.1 adaptations #472

Merged
merged 32 commits into from
May 19, 2023
Merged

0.11.1 adaptations #472

merged 32 commits into from
May 19, 2023

Conversation

mikiw
Copy link
Contributor

@mikiw mikiw commented May 16, 2023 •
edited
Loading

Usage related changes

  • Adaptations to cairo-lang v0.11.1.1
  • DEFAULT_GAS_PRICE was set to 10**8 - the same as in starkware codebase

Checklist:

  • Applied formatting - ./scripts/format.sh
  • No linter errors - ./scripts/lint.sh
  • Performed code self-review
  • Rebased to the last commit of the target branch (or merged it into my branch)
  • Documented the changes
  • Linked the issues which this PR resolves
  • Updated the tests
  • All tests are passing - ./scripts/test.sh

@mikiw mikiw marked this pull request as draft May 16, 2023 07:05
@mikiw mikiw linked an issue May 16, 2023 that may be closed by this pull request
0.11.1a0 adaptations #464
Closed
@mikiw mikiw changed the title Adapt 0.11.1 0.11.1a0 adaptations May 16, 2023
@FabijanC FabijanC changed the title 0.11.1a0 adaptations 0.11.1 adaptations May 17, 2023
@piwonskp piwonskp mentioned this pull request May 17, 2023
Merged
@mikiw mikiw marked this pull request as ready for review May 18, 2023 10:45
@mikiw mikiw requested a review from FabijanC May 18, 2023 11:08
FabijanC
FabijanC suggested changes May 18, 2023
View reviewed changes
starknet_devnet/postman_wrapper.py Show resolved Hide resolved
starknet_devnet/general_config.py Show resolved Hide resolved
starknet_devnet/starknet_wrapper.py Outdated Show resolved Hide resolved
starknet_devnet/constants.py Show resolved Hide resolved
@mikiw mikiw requested a review from FabijanC May 19, 2023 07:34
@mikiw
Copy link
Contributor Author

mikiw commented May 19, 2023

after merge test are failing, I'm on it

@FabijanC FabijanC merged commit eed2ecc into master May 19, 2023
@FabijanC FabijanC deleted the adapt-0.11.1 branch May 19, 2023 16:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@FabijanC FabijanC FabijanC 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.

0.11.1a0 adaptations
2 participants
@mikiw @FabijanC