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

Move Context from GSN to utils directory #2453

Merged
merged 6 commits into from
Jan 7, 2021

Conversation

Amxx
Copy link
Collaborator

@Amxx Amxx commented Jan 5, 2021

Proposal to move the Context contract to the contracts/utils folder

@frangio frangio changed the title context moved from gsn to utils Move Context from GSN to utils directory Jan 5, 2021
@frangio
Copy link
Contributor

frangio commented Jan 5, 2021

To provide some context: we want to move this file out of the GSN directory to make it clear that Context itself has nothing to do with GSN. It enables opting in to GSN if a developer wants to do so.

Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

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

Thanks!

For backwards compatibility we need to keep a file contracts/GSN/Context.sol that just imports the file from the new location:

// SPDX-License-Identifier: MIT

pragma solidity >=0.6.0 <0.8.0;

import '../utils/Context.sol';

@frangio frangio merged commit 318c4b4 into OpenZeppelin:master Jan 7, 2021
@Amxx Amxx deleted the refactor/context-in-utils branch January 7, 2021 16:07
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.

2 participants