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

Add @openzeppelin/contracts as alternative package name #1840

Merged
merged 8 commits into from
Jul 26, 2019

Conversation

spalladino
Copy link
Contributor

This PR renames the package to @openzeppelin/contracts, changes all links in documentation (including links to the old repository name), and moves all contract folders to the project root to avoid awkward import paths like import "@openzeppelin/contracts/contracts/ownership/Ownable.sol";.

@spalladino spalladino requested a review from frangio July 23, 2019 19:56
@nventuro
Copy link
Contributor

Before we update the readme with the new package name, we should publish it on npm. It will probably be easier to do that by merging this into release-v2.3.0, publishing, and then merging back into master.

@frangio frangio force-pushed the feature/rename-to-openzeppelin-contracts branch from c59ba7d to 3385623 Compare July 26, 2019 16:29
@frangio frangio changed the title Rename to @openzeppelin/contracts Add @openzeppelin/contracts as alternative package name Jul 26, 2019
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 @spalladino!

@frangio frangio merged commit b8c8308 into master Jul 26, 2019
@frangio frangio deleted the feature/rename-to-openzeppelin-contracts branch July 26, 2019 19:06
frangio pushed a commit that referenced this pull request Jul 26, 2019
* Rename package and repository name from docs and scripts

* undo root package rename

* add @OpenZeppelin/contracts as subpackage with release automation

* synchronize @OpenZeppelin/contracts version

* remove private field from package.json

* make file patterns absolute

* change wording of a comment

* use a saner version script

(cherry picked from commit b8c8308)
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.

3 participants