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

chore: Bump @metamask/permission-log-controller to v3.0.1 #28747

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Nov 26, 2024

Description

Update @metamask/permission-log-controller from v2.0.1 to v3.0.1. No breaking changes that impact the extension.

Changelog: https://github.com/MetaMask/core/blob/main/packages/permission-log-controller/CHANGELOG.md#301

Open in GitHub Codespaces

Related issues

Relates to https://github.com/MetaMask/MetaMask-planning/issues/3568

Manual testing steps

N/A

Screenshots/Recordings

N/A

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@Gudahtt
Copy link
Member Author

Gudahtt commented Nov 26, 2024

@metamaskbot update-policies

Copy link

socket-security bot commented Nov 26, 2024

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

Package New capabilities Transitives Size Publisher
npm/@metamask/[email protected] None +2 435 kB metamaskbot

🚮 Removed packages: npm/@metamask/[email protected]

View full report↗︎

@metamaskbot
Copy link
Collaborator

Policies updated.
👀 Please review the diff for suspicious new powers.

🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff

@Gudahtt Gudahtt marked this pull request as ready for review November 26, 2024 22:55
@Gudahtt Gudahtt requested review from a team as code owners November 26, 2024 22:55
@metamaskbot
Copy link
Collaborator

Builds ready [3cd1642]
Page Load Metrics (2034 ± 70 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint18262467203715273
domContentLoaded17612397199014268
load18232464203414770
domInteractive29165483014
backgroundConnect879392613
firstReactRender20108523316
getState877272211
initialActions01000
loadScripts12511836146813866
setupStore777262512
uiStartup202027392351210101
Bundle size diffs [🚀 Bundle size reduced!]
  • background: -11.84 KiB (-0.20%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

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.

LGTM.

Gudahtt and others added 2 commits November 26, 2024 20:55
Update `@metamask/permission-log-controller` from v2.0.1 to v3.0.1. No
breaking changes that impact the extension.

Changelog: https://github.com/MetaMask/core/blob/main/packages/permission-log-controller/CHANGELOG.md#301
@Gudahtt Gudahtt force-pushed the bump-permission-log-controller-v3 branch from 3cd1642 to c8d1747 Compare November 27, 2024 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants