-
-
Notifications
You must be signed in to change notification settings - Fork 97
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
v0.16.0 #189
v0.16.0 #189
Conversation
136c6bf
to
7a44cf5
Compare
7a44cf5
to
f8e5846
Compare
CHANGELOG.md
Outdated
## [0.16.0] | ||
### Changed | ||
- **BREAKING:** Refactor code to split keyring from bridge logic. ([#156](https://github.com/MetaMask/eth-ledger-bridge-keyring/pull/156)) | ||
- Migrate code to TypeScript. ([#174](https://github.com/MetaMask/eth-ledger-bridge-keyring/pull/174)) |
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.
This should probably be two entries: one for the added types (non-breaking), and one for the change from default export to named export (breaking).
Anything else worth noting here @mikesposito ?
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.
Done.
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "@metamask/eth-ledger-bridge-keyring", | |||
"version": "0.15.0", | |||
"version": "0.16.0", |
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.
Thoughts on making this v1 instead? @gantunesr and I are in agreement that it seems appropriate
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.
I've bumped the version, but I cannot change the branch name.
I've created a new PR with the correct branch name and all the changes discussed: #191
c230a6a
to
117719f
Compare
Superceded by #191 |
Changed