Skip to content

devDeps: @metamask/eslint-config*->12.1.0 - #1740

Merged
legobeat merged 11 commits into
MetaMask:mainfrom
legobeat:devDeps-typescript-eslint-configs
Oct 2, 2023
Merged

devDeps: @metamask/eslint-config*->12.1.0#1740
legobeat merged 11 commits into
MetaMask:mainfrom
legobeat:devDeps-typescript-eslint-configs

Conversation

@legobeat

Copy link
Copy Markdown
Contributor

Explanation

  • Bump @metamask/eslint-config* packages to latest.
    • Bump typedoc,typedoc-plugin-missing-exports to satisfy peerDeps

References

Related

Changelog

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate

@socket-security

socket-securityBot commented Sep 27, 2023

Copy link
Copy Markdown

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

PackagesVersionNew capabilitiesTransitivesSizePublisher
typedoc-plugin-missing-exports0.23.0...2.1.0None+25/-5030.5 MBgerrit0
eslint-plugin-import2.28.1...2.26.0None+2/-491.12 MBljharb
typedoc0.23.28...0.24.8None+0/-251.22 MBtypedoc-bot

@legobeat
legobeat marked this pull request as ready for review September 27, 2023 23:13
@legobeat
legobeat requested review from a team as code ownersSeptember 27, 2023 23:13
@legobeatlegobeat added the dependencies Pull requests that update a dependency file label Sep 27, 2023
Gudahtt
Gudahtt previously approved these changes Sep 27, 2023

@GudahttGudahtt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@legobeat

Copy link
Copy Markdown
ContributorAuthor

@MetaMask/snaps-devs needs your approval :)

@legobeat

legobeat commented Sep 28, 2023

Copy link
Copy Markdown
ContributorAuthor

Updated with bumping range for @typescript-eslint/eslint-plugin and @typescript-eslint/parser. 394e3234cd7ac0d7503fa4b3e1f3af949cb09786

These were already at the most recent versions in lockfile so this does not make a difference in practice but done for sake of smooth migrations of other packages into this monorepo.

Gudahtt
Gudahtt previously approved these changes Sep 28, 2023
Mrtenz
Mrtenz previously approved these changes Sep 28, 2023

@MrtenzMrtenz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Technically unrelated to your change, but we should fix this.

Comment threadpackages/transaction-controller/src/TransactionController.test.ts Outdated
@legobeat
legobeat dismissed stale reviews from Mrtenz and Gudahtt via 8826e02September 28, 2023 21:47
MajorLift
MajorLift previously approved these changes Sep 28, 2023
@legobeat
legobeatforce-pushed the devDeps-typescript-eslint-configs branch from 8826e02 to b61aa89CompareSeptember 29, 2023 01:52
@legobeat
legobeatforce-pushed the devDeps-typescript-eslint-configs branch from 0c70e96 to 37a3ff7CompareSeptember 29, 2023 04:00
MajorLift
MajorLift previously approved these changes Sep 29, 2023
@legobeat
legobeat requested review from a teamSeptember 29, 2023 04:39
Mrtenz
Mrtenz previously approved these changes Sep 29, 2023
@legobeat
legobeat dismissed stale reviews from MajorLift and Mrtenz via aece1e0September 30, 2023 01:50
@legobeat
legobeatforce-pushed the devDeps-typescript-eslint-configs branch from 421869f to aece1e0CompareSeptember 30, 2023 01:50
@legobeat
legobeat requested review from a team, MajorLift and MrtenzSeptember 30, 2023 01:58
@legobeat
legobeat requested review from a teamOctober 2, 2023 00:26
@legobeat
legobeat merged commit 53c6b76 into MetaMask:mainOct 2, 2023
@legobeat
legobeat deleted the devDeps-typescript-eslint-configs branch October 2, 2023 00:26
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependenciesPull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@legobeat@Gudahtt@Mrtenz@MajorLift