Skip to content
This repository was archived by the owner on Oct 7, 2024. It is now read-only.

Conversation

@github-actions
Copy link
Contributor

This is the release candidate for version 11.0.0.

@legobeat

This comment was marked as resolved.

@legobeat legobeat marked this pull request as ready for review May 11, 2023 21:36
@legobeat legobeat requested a review from a team as a code owner May 11, 2023 21:36
CHANGELOG.md Outdated
Comment on lines 12 to 15
- docs: Update README ([#209](https://github.com/MetaMask/KeyringController/pull/209))
- deps: Unpin and bump @metamask/eth-sig-utils@5.0.2->^5.1.0 ([#224](https://github.com/MetaMask/KeyringController/pull/224))
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want to remove these last two entries since they aren't user-facing?

Copy link
Member

Choose a reason for hiding this comment

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

This entry is; this lets users of this package use later versions of eth-sig-util. Agreed on the readme update though

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, fair.

github-actions and others added 4 commits May 12, 2023 19:58
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
@legobeat legobeat requested review from Gudahtt and mcmire May 12, 2023 19:58
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@legobeat legobeat merged commit 2746ae1 into main May 12, 2023
@legobeat legobeat deleted the release/11.0.0 branch May 12, 2023 21:59
@Gudahtt
Copy link
Member

Gudahtt commented May 12, 2023

Had to reject the deployment, the code was missing! The GitHub actions workflows weren't updated after the TypeScript migration to include a build before publish.

We should revert this, then try again after fixing the workflow

Gudahtt added a commit that referenced this pull request May 12, 2023
This reverts commit 2746ae1.
@Gudahtt Gudahtt mentioned this pull request May 12, 2023
Gudahtt added a commit that referenced this pull request May 12, 2023
@gantunesr
Copy link
Member

gantunesr commented May 12, 2023

@Gudahtt @legobeat I'm working on some fixes I spotted last week in the TS refactor. I had planned to work on it next week, sorry for not giving a heads up

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants