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

Bump @metamask/eth-keyring-controller from 13.0.1 to 14.0.0 #1771

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 4, 2023

⚠️ Dependabot is rebasing this PR ⚠️

Rebasing might not happen immediately, so don't worry if this takes some time.

Note: if you make any changes to this PR yourself, they will take precedence over the rebase.


Bumps @metamask/eth-keyring-controller from 13.0.1 to 14.0.0.

Release notes

Sourced from @​metamask/eth-keyring-controller's releases.

14.0.0

Changed

  • BREAKING: Bump @metamask/eth-sig-util from ^6.0.0 to ^7.0.0 (#269)
  • BREAKING: Bump @metamask/eth-hd-keyring from ^6.0.0 to ^7.0.1 (#275)
  • BREAKING: Bump @metamask/eth-simple-keyring from ^5.0.0 to ^6.0.0 (#273)
Changelog

Sourced from @​metamask/eth-keyring-controller's changelog.

[14.0.0]

Changed

  • BREAKING: Bump @metamask/eth-sig-util from ^6.0.0 to ^7.0.0 (#269)
  • BREAKING: Bump @metamask/eth-hd-keyring from ^6.0.0 to ^7.0.1 (#275)
  • BREAKING: Bump @metamask/eth-simple-keyring from ^5.0.0 to ^6.0.0 (#273)
Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested a review from a team as a code owner October 4, 2023 12:20
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Oct 4, 2023
@socket-security
Copy link

socket-security bot commented Oct 4, 2023

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@metamask/eth-keyring-controller 14.0.0 None +4 360 kB metamaskbot

@legobeat
Copy link
Contributor

@dependabot recreate

@legobeat

This comment was marked as resolved.

@mcmire
Copy link
Contributor

mcmire commented Oct 26, 2023

@dependabot recreate

1 similar comment
@legobeat
Copy link
Contributor

@dependabot recreate

@legobeat legobeat force-pushed the dependabot/npm_and_yarn/metamask/eth-keyring-controller-14.0.0 branch 3 times, most recently from 8127e9e to a19c990 Compare October 27, 2023 11:30
dependabot bot and others added 2 commits October 27, 2023 11:31
Bumps [@metamask/eth-keyring-controller](https://github.com/MetaMask/KeyringController) from 13.0.1 to 14.0.0.
- [Release notes](https://github.com/MetaMask/KeyringController/releases)
- [Changelog](https://github.com/MetaMask/KeyringController/blob/main/CHANGELOG.md)
- [Commits](MetaMask/KeyringController@v13.0.1...v14.0.0)

---
updated-dependencies:
- dependency-name: "@metamask/eth-keyring-controller"
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@legobeat legobeat force-pushed the dependabot/npm_and_yarn/metamask/eth-keyring-controller-14.0.0 branch from a19c990 to 9c39d5f Compare October 27, 2023 11:31
Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Looks good.

@@ -31,7 +31,7 @@
"dependencies": {
"@keystonehq/metamask-airgapped-keyring": "^0.13.1",
"@metamask/base-controller": "^3.2.3",
"@metamask/eth-keyring-controller": "^13.0.1",
"@metamask/eth-keyring-controller": "^14.0.0",
Copy link
Contributor

@mcmire mcmire Oct 27, 2023

Choose a reason for hiding this comment

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

Just checking the dependency tree to check whether this would introduce breaking changes, and it seems that it would not. The updates seem to mainly be to unrelated parts of @metamask/utils. The bump to ethereum-cryptography could be breaking but doesn't seem that way.

Would you agree?

Copy link
Contributor

Choose a reason for hiding this comment

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

Could it exacerbate the issue here? #1868

Copy link
Contributor

Choose a reason for hiding this comment

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

Reflects my experience as well.

@legobeat legobeat merged commit c7b8f71 into main Oct 27, 2023
115 checks passed
@legobeat legobeat deleted the dependabot/npm_and_yarn/metamask/eth-keyring-controller-14.0.0 branch October 27, 2023 22:48
mcmire pushed a commit that referenced this pull request Nov 2, 2023
[//]: # (dependabot-start)
⚠️  **Dependabot is rebasing this PR** ⚠️ 

Rebasing might not happen immediately, so don't worry if this takes some
time.

Note: if you make any changes to this PR yourself, they will take
precedence over the rebase.

---

[//]: # (dependabot-end)

Bumps
[@metamask/eth-keyring-controller](https://github.com/MetaMask/KeyringController)
from 13.0.1 to 14.0.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/MetaMask/KeyringController/releases"><code>@​metamask/eth-keyring-controller</code>'s
releases</a>.</em></p>
<blockquote>
<h2>14.0.0</h2>
<h3>Changed</h3>
<ul>
<li><strong>BREAKING:</strong> Bump <code>@metamask/eth-sig-util</code>
from <code>^6.0.0</code> to <code>^7.0.0</code> (<a
href="https://redirect.github.com/MetaMask/KeyringController/pull/269">#269</a>)</li>
<li><strong>BREAKING:</strong> Bump
<code>@metamask/eth-hd-keyring</code> from <code>^6.0.0</code> to
<code>^7.0.1</code> (<a
href="https://redirect.github.com/MetaMask/KeyringController/pull/275">#275</a>)</li>
<li><strong>BREAKING:</strong> Bump
<code>@metamask/eth-simple-keyring</code> from <code>^5.0.0</code> to
<code>^6.0.0</code> (<a
href="https://redirect.github.com/MetaMask/KeyringController/pull/273">#273</a>)</li>
</ul>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/MetaMask/KeyringController/blob/main/CHANGELOG.md"><code>@​metamask/eth-keyring-controller</code>'s
changelog</a>.</em></p>
<blockquote>
<h2>[14.0.0]</h2>
<h3>Changed</h3>
<ul>
<li><strong>BREAKING:</strong> Bump <code>@metamask/eth-sig-util</code>
from <code>^6.0.0</code> to <code>^7.0.0</code> (<a
href="https://redirect.github.com/MetaMask/KeyringController/pull/269">#269</a>)</li>
<li><strong>BREAKING:</strong> Bump
<code>@metamask/eth-hd-keyring</code> from <code>^6.0.0</code> to
<code>^7.0.1</code> (<a
href="https://redirect.github.com/MetaMask/KeyringController/pull/275">#275</a>)</li>
<li><strong>BREAKING:</strong> Bump
<code>@metamask/eth-simple-keyring</code> from <code>^5.0.0</code> to
<code>^6.0.0</code> (<a
href="https://redirect.github.com/MetaMask/KeyringController/pull/273">#273</a>)</li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/MetaMask/KeyringController/commit/4943da5ed630cac2ab6cfa0160a5d266e60d131d"><code>4943da5</code></a>
14.0.0 (<a
href="https://redirect.github.com/MetaMask/KeyringController/issues/265">#265</a>)</li>
<li><a
href="https://github.com/MetaMask/KeyringController/commit/4c785f490f5ae00f81e32d84e73806e7dd45aed2"><code>4c785f4</code></a>
devDeps: bump <code>@metamask/eslint-config*</code> to v12 (<a
href="https://redirect.github.com/MetaMask/KeyringController/issues/262">#262</a>)</li>
<li><a
href="https://github.com/MetaMask/KeyringController/commit/256c540cabaa39dce9bb787ac5cd9601b3f8c029"><code>256c540</code></a>
deps: <code>@​metamask/eth-hd-keyring</code>@^6.0.0-&gt;^7.0.1 (<a
href="https://redirect.github.com/MetaMask/KeyringController/issues/275">#275</a>)</li>
<li><a
href="https://github.com/MetaMask/KeyringController/commit/4e4c471d441e30802e0b44c6b006f60ff3728282"><code>4e4c471</code></a>
deps: <code>@​metamask/eth-simple-keyring</code>@^5.0.0-&gt;^6.0.0 (<a
href="https://redirect.github.com/MetaMask/KeyringController/issues/273">#273</a>)</li>
<li><a
href="https://github.com/MetaMask/KeyringController/commit/8d30a98176fe04072b694ede2a0dfc4b6838aafd"><code>8d30a98</code></a>
Bump <code>@​metamask/auto-changelog</code> from 3.2.0 to 3.3.0 (<a
href="https://redirect.github.com/MetaMask/KeyringController/issues/278">#278</a>)</li>
<li><a
href="https://github.com/MetaMask/KeyringController/commit/24f93d9742b2b36e33b23ea3abb2de6f8f5d9088"><code>24f93d9</code></a>
Bump <code>@​metamask/eth-sig-util</code> from 6.0.0 to 7.0.0 (<a
href="https://redirect.github.com/MetaMask/KeyringController/issues/269">#269</a>)</li>
<li>See full diff in <a
href="https://github.com/MetaMask/KeyringController/compare/v13.0.1...v14.0.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=@metamask/eth-keyring-controller&package-manager=npm_and_yarn&previous-version=13.0.1&new-version=14.0.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: legobt <[email protected]>
Co-authored-by: legobeat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants