Skip to content
This repository was archived by the owner on Dec 23, 2025. It is now read-only.

fix: openzeppelinV3 should be just openzeppelin #84

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

fix: openzeppelinV3 should be just openzeppelin #84

lbertenasco wants to merge 2 commits into from

Conversation

@lbertenasco
Copy link
Contributor

@lbertenasco lbertenasco commented Jan 20, 2021

No description provided.

@lbertenasco lbertenasco requested a review from fubuloubu January 20, 2021 18:19
@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

lbertenasco commented Jan 20, 2021

Update the contract?

No file is using V3 lol

@fubuloubu
Copy link
Member

fubuloubu commented Jan 20, 2021

Update the contract?

No file is using V3 lol

So why have it?

@lbertenasco
Copy link
Contributor Author

lbertenasco commented Jan 20, 2021

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@fubuloubu fubuloubu fubuloubu requested changes

Reviewers whose approvals may not affect merge requirements

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.

3 participants

@lbertenasco @fubuloubu