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

Update oz account #137

Merged
merged 7 commits into from
Jun 20, 2022
Merged

Update oz account #137

merged 7 commits into from
Jun 20, 2022

Conversation

badurinantun
Copy link
Contributor

@badurinantun badurinantun commented Jun 20, 2022 •
edited by FabijanC
Loading

Usage related changes

Development related changes

  • Use GH commit hash for the Account version

Checklist:

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

@badurinantun badurinantun self-assigned this Jun 20, 2022
@badurinantun badurinantun marked this pull request as ready for review June 20, 2022 10:31
@badurinantun badurinantun requested a review from FabijanC June 20, 2022 10:31
FabijanC
FabijanC reviewed Jun 20, 2022
View reviewed changes
Copy link
Collaborator

@FabijanC FabijanC 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, just left a comment regarding a print in a test.

test/test_account.py Outdated Show resolved Hide resolved
@FabijanC FabijanC merged commit 90964bf into master Jun 20, 2022
@FabijanC FabijanC deleted the update-oz-account branch June 20, 2022 13:13
FabijanC added a commit that referenced this pull request Jun 20, 2022
@FabijanC
Hotfix of #137
2cce5d9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@FabijanC FabijanC FabijanC left review comments

Assignees

@badurinantun badurinantun

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update open zeppelin account contracts
2 participants
@badurinantun @FabijanC