Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Add state variables default initialisation to _initializate() #239

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

jmonteer
Copy link
Contributor

@jmonteer jmonteer commented Mar 11, 2021

closes #240

@jmonteer jmonteer changed the title added state var default init to _initialize method Add state variables default initialisation to _initializate() Mar 11, 2021
@jmonteer jmonteer changed the title Add state variables default initialisation to _initializate() fix: Add state variables default initialisation to _initializate() Mar 11, 2021
@jmonteer jmonteer marked this pull request as draft March 11, 2021 11:10
@jmonteer jmonteer force-pushed the jmonteer/initialize_clone_fix branch from b346099 to 07604bd Compare March 11, 2021 13:10
@jmonteer jmonteer marked this pull request as ready for review March 11, 2021 13:49
@fubuloubu fubuloubu merged commit 6ff96e6 into yearn:master Mar 11, 2021
orbxball pushed a commit to orbxball/yearn-vaults that referenced this pull request Mar 22, 2021
orbxball pushed a commit to orbxball/yearn-vaults that referenced this pull request Aug 8, 2021
sambacha pushed a commit to sambacha/yearn-vaults that referenced this pull request Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cloning a strategy leaves hardcoded state variables uninitialized
2 participants