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
Rename to @openzeppelin/contracts-ethereum-package #54
Conversation
Not sure we want to merge into master yet since this includes links to repos that don't yet exist, to the old zos docsite, etc. |
package.json
Outdated
"name": "openzeppelin-eth", | ||
"version": "2.2.0", | ||
"name": "@openzeppelin/contracts-ethereum-package", | ||
"version": "2.2.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be a release candidate, if the dependencies we're using are also release candidates?
LGTM. I have no idea why the CI is failing, but it was failing before too. 🙂 I'm contacting Travis support to migrate the repo to from travis-ci.org to travis-ci.com. |
@nico I'd merge either way, so we ensure we keep working on the renamed version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed the PR base to master
.
I'm opening this PR against a new
@openzeppelin
branch, since I'm not sure if we want to merge this into master right away. I don't see any reason not to though!