Skip to content
This repository has been archived by the owner on Nov 22, 2021. It is now read-only.

fix: openzeppelinV3 should be just openzeppelin #84

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

fix: openzeppelinV3 should be just openzeppelin #84

wants to merge 2 commits into from

Conversation

lbertenasco
Copy link
Contributor

No description provided.

@lbertenasco lbertenasco self-assigned this Jan 20, 2021
fubuloubu
fubuloubu suggested changes Jan 20, 2021
View reviewed changes
Copy link
Member

@fubuloubu fubuloubu left a comment

Choose a reason for hiding this comment

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

Update the contract?

@lbertenasco
Copy link
Contributor Author

Update the contract?

No file is using V3 lol

@fubuloubu
Copy link
Member

Update the contract?

No file is using V3 lol

So why have it?

@lbertenasco
Copy link
Contributor Author

Update the contract?

No file is using V3 lol

So why have it?

To avoid new implementations to use it with the V3 suffix :P

@saltyfacu saltyfacu added this to In progress in Vaults & Strategies Feb 2, 2021
@saltyfacu saltyfacu removed this from In progress in Vaults & Strategies Feb 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@fubuloubu fubuloubu fubuloubu requested changes

At least 1 approving review is required to merge this pull request.

Assignees

@lbertenasco lbertenasco

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@lbertenasco @fubuloubu