Skip to content

deps: force js-sha3 to ^0.9.2 by resolutions#20921

Merged
HowardBraham merged 2 commits intoMetaMask:developfrom
legobeat:deps-dedupe-js-sha3
Jan 31, 2024
Merged

deps: force js-sha3 to ^0.9.2 by resolutions#20921
HowardBraham merged 2 commits intoMetaMask:developfrom
legobeat:deps-dedupe-js-sha3

Conversation

@legobeat
Copy link
Copy Markdown
Contributor

@legobeat legobeat commented Sep 16, 2023

Explanation

Forces legacy transitive dependencies on js-sha3 to latest via yarn resolution.

Manual Testing Steps

Pre-merge author checklist

  • I've clearly explained:
    • What problem this PR is solving
    • How this problem was solved
    • How reviewers can test my changes
  • Sufficient automated test coverage has been added

Pre-merge reviewer checklist

  • Manual testing (e.g. pull and build branch, run in browser, test code being changed)
  • PR is linked to the appropriate GitHub issue
  • IF this PR fixes a bug in the release milestone, add this PR to the release milestone

If further QA is required (e.g. new feature, complex testing steps, large refactor), add the Extension QA Board label.

In this case, a QA Engineer approval will be be required.

@github-actions
Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@legobeat legobeat added dependencies Pull requests that update a dependency file team-application-security Application security team labels Sep 16, 2023
@socket-security
Copy link
Copy Markdown

socket-security bot commented Sep 16, 2023

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

@legobeat legobeat force-pushed the deps-dedupe-js-sha3 branch 2 times, most recently from b573eef to a02b433 Compare September 16, 2023 22:28
@legobeat legobeat changed the title deps: force js-sha3 to ^0.9.1 by resolutions deps: force js-sha3 to ^0.9.2 by resolutions Sep 20, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 20, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (50c0f34) 68.06% compared to head (e441709) 68.05%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #20921      +/-   ##
===========================================
- Coverage    68.06%   68.05%   -0.00%     
===========================================
  Files         1083     1083              
  Lines        42520    42520              
  Branches     11367    11367              
===========================================
- Hits         28937    28936       -1     
- Misses       13583    13584       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@legobeat legobeat marked this pull request as ready for review September 20, 2023 08:23
@legobeat legobeat requested review from a team as code owners September 20, 2023 08:23
@legobeat legobeat requested review from a team September 27, 2023 23:01
@legobeat legobeat force-pushed the deps-dedupe-js-sha3 branch 3 times, most recently from f5f8e76 to ef2b6ca Compare October 16, 2023 11:33
Mrtenz
Mrtenz previously approved these changes Oct 17, 2023
@legobeat legobeat force-pushed the deps-dedupe-js-sha3 branch 5 times, most recently from aa64aed to 50827eb Compare October 24, 2023 10:15
@legobeat legobeat force-pushed the deps-dedupe-js-sha3 branch 4 times, most recently from 671b9a9 to cf93671 Compare November 1, 2023 11:20
NEllusion
NEllusion previously approved these changes Jan 15, 2024
@legobeat legobeat dismissed stale reviews from NEllusion and mcmire via f9c8302 January 16, 2024 23:00
@legobeat legobeat force-pushed the deps-dedupe-js-sha3 branch 3 times, most recently from ac10620 to 1b1aecb Compare January 18, 2024 14:54
@legobeat
Copy link
Copy Markdown
Contributor Author

Rebased on develop without further changes to resolve merge conflict.

@legobeat legobeat force-pushed the deps-dedupe-js-sha3 branch 8 times, most recently from 659ae50 to 47cf208 Compare January 22, 2024 08:57
@legobeat legobeat requested review from mcmire and mikesposito and removed request for mikesposito January 22, 2024 08:57
@legobeat legobeat force-pushed the deps-dedupe-js-sha3 branch 4 times, most recently from f30749d to 64a82e1 Compare January 23, 2024 10:34
@legobeat legobeat requested review from a team and NEllusion January 23, 2024 11:20
@legobeat
Copy link
Copy Markdown
Contributor Author

legobeat commented Jan 30, 2024

test-e2e-chrome-mv3 failing is unrelated to this change and addressed in #22660

@metamaskbot
Copy link
Copy Markdown
Collaborator

No release label on PR. Adding release label release-11.11.0 on PR, as PR was added to branch 11.11.0 when release was cut.

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

Labels

dependencies Pull requests that update a dependency file release-11.11.0 Issue or pull request that will be included in release 11.11.0 team-application-security Application security team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants