-
-
Notifications
You must be signed in to change notification settings - Fork 7
chore(deps-dev): bump @metamask/eslint-config from 14.1.0 to 15.0.0 #854
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(deps-dev): bump @metamask/eslint-config from 14.1.0 to 15.0.0 #854
Conversation
📖 Storybook Preview |
246a95a to
cb65979
Compare
📖 Storybook Preview |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
📖 Storybook Preview |
📖 Storybook Preview |
a17d5c6 to
87600c4
Compare
📖 Storybook Preview |
87600c4 to
115f472
Compare
| // TODO: Re-enable these rules. Enabling them with error suppression | ||
| // breaks `--fix`, because the autofixer for these rules do not work very | ||
| // well. | ||
| 'jsdoc/require-jsdoc': 'off', | ||
| 'jsdoc/check-tag-names': 'off', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignores all base files
| // TODO: Re-enable these rules. Enabling them with error suppression | ||
| // breaks `--fix`, because the autofixer for these rules do not work very | ||
| // well. | ||
| 'jsdoc/require-jsdoc': 'off', | ||
| 'jsdoc/check-tag-names': 'off', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignores for typescript files
📖 Storybook Preview |
| 'scripts/create-package/**/*.ts', | ||
| ], | ||
| extends: [nodejs], | ||
| rules: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignoring jsdoc require rules as we have over 900+ violations an the auto fix doesn't really work. Core does the same thing https://github.com/MetaMask/core/pull/7305/files?file-filters%5B%5D=.mjs#diff-9601a8f6c734c2001be34a2361f76946d19a39a709b5e8c624a2a5a0aade05f2
What yarn lint:fix looks like without disabling these rules and lint still fails
📖 Storybook Preview |
## **Description** This PR releases version 17.0.0 of the MetaMask Design System, featuring important fixes for mobile font compatibility, the addition of the ButtonHero component to React, export fixes for React Native, and various dependency updates including ESLint configuration upgrades. ## **Included PRs** #836, #837, #838, #840, #843, #845, #846, #847, #848, #850, #851, #852, #853, #854, #855, #856, #857, #858, #859, #861, #862, #863, #864 ## **Manual testing steps** 1. Check package.json version bumps align with included changes 2. Check changelog accurately reflects the release ## **Pre-merge author checklist** - [x] I've followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) - [x] I've completed the PR template to the best of my ability - [x] I've included tests if applicable - [x] I've documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [x] I've applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. ## **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. <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Release 17.0.0 introducing React `ButtonHero`, RN font naming changes and TWRNC preset font family rename (both breaking), plus RN export fix and dependency updates. > > - **Release 17.0.0** > - Bump root `package.json` to `17.0.0`. > - **React (`@metamask/[email protected]`)** > - Add `ButtonHero` component for prominent CTAs. > - Update `@metamask/utils` peer to `^11.8.1`. > - **React Native (`@metamask/[email protected]`)** > - BREAKING: Rename font files to hyphenated PostScript format for iOS Metro compatibility. > - Export missing `TextButtonSize` enum. > - Peer: require `@metamask/design-system-twrnc-preset@^0.3.0`; update `@metamask/utils` peer. > - **TWRNC preset (`@metamask/[email protected]`)** > - BREAKING: Rename font family names to hyphenated PostScript format for iOS/expo-font. > - **Changelogs** > - Update `CHANGELOG.md` links and entries for all packages. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 9a90311. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
Bumps @metamask/eslint-config from 14.1.0 to 15.0.0.
Release notes
Sourced from
@metamask/eslint-config's releases.... (truncated)
Commits
0b8cf6dRelease/15.0.0 (#431)9568eb3fix: Disablejsdoc/check-indentationto fix bullet list formatting (#430)7837174feat: Loosen@typescript-eslint/naming-conventionfor object literal proper...cf93ef1fix: Disableimport-x/no-duplicatesin TypeScript files (#427)f3246f7chore: Disable redudant lint rule (#425)383fbecfeat: Disable@typescript-eslint/no-unnecessary-type-arguments(#426)a0e0bc8feat: Convert all warnings to errors (#424)b243e90chore: Increase minimumtypescript-eslintversion (#423)b116974Allow remaining TypeScript 5.x versions (#422)c70a3ffBREAKING: Updatejsdoc/require-jsdocto require documentation for more thin...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 rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill 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 versionwill 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 dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Note
Upgrade ESLint config to v15 and adjust local lint rules by disabling stricter JSDoc checks to keep autofix stable.
@metamask/eslint-configfrom^14.1.0to^15.0.0inpackage.json(lockfile updated).eslint.config.mjs):jsdoc/require-jsdocandjsdoc/check-tag-namesto avoid autofix issues introduced by v15.Written by Cursor Bugbot for commit 61c59be. This will update automatically on new commits. Configure here.