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

Update packages and configs for eslint and prettier #1683

Closed
mikesposito opened this issue Sep 18, 2023 · 0 comments · Fixed by MetaMask/eth-json-rpc-provider#28
Closed

Update packages and configs for eslint and prettier #1683

mikesposito opened this issue Sep 18, 2023 · 0 comments · Fixed by MetaMask/eth-json-rpc-provider#28
Assignees

Comments

@mikesposito
Copy link
Member

Currently, the core monorepo is using these package versions related to eslint and prettier:

// ..
"@metamask/eslint-config": "^12.0.0",
"@metamask/eslint-config-jest": "^12.0.0",
"@metamask/eslint-config-nodejs": "^12.0.0",
"@metamask/eslint-config-typescript": "^12.0.0"
"eslint": "^8.44.0",
"eslint-config-prettier": "^8.5.0",
"eslint-import-resolver-typescript": "^2.5.0",
"eslint-plugin-import": "^2.26.0",
"eslint-plugin-jest": "^27.1.5",
"eslint-plugin-jsdoc": "^39.3.3",
"eslint-plugin-n": "^15.7.0",
"eslint-plugin-prettier": "^4.2.1",
"eslint-plugin-promise": "^6.1.1",
"prettier": "^2.7.1",
"prettier-plugin-packagejson": "^2.2.17",
// ..
// See: https://github.com/MetaMask/core/blob/main/package.json

To migrate @metamask/eth-json-rpc-provider into the core monorepo smoothly, we should match the package versions of the core monorepo, updating the ones that differ and updating the config when needed (.prettierrc*, .eslintrc.js etc.. see the module template for reference).

MajorLift added a commit that referenced this issue Sep 22, 2023
…norepo

- added: `eslint-plugin-promise`
- removed: `eslint-plugin-node`

Closes #1683
MajorLift added a commit to MetaMask/eth-json-rpc-provider that referenced this issue Sep 25, 2023
…norepo

- added: `eslint-plugin-promise`
- removed: `eslint-plugin-node`

See: MetaMask/core#1683
MajorLift added a commit to MetaMask/eth-json-rpc-provider that referenced this issue Sep 25, 2023
- added `eslint-import-resolver-typescript` v2.5.0 as dependency

See: MetaMask/core#1683
MajorLift added a commit that referenced this issue Sep 27, 2023
…norepo

- added: `eslint-plugin-promise`
- removed: `eslint-plugin-node`

See: #1683
MajorLift added a commit that referenced this issue Sep 27, 2023
- added `eslint-import-resolver-typescript` v2.5.0 as dependency

See: #1683
MajorLift added a commit to MetaMask/eth-json-rpc-provider that referenced this issue Sep 28, 2023
…of migration (#28)

## Description
- This PR aligns the ESLint and Prettier packages and configs in this repo with the core monorepo.
- This is a preparation step for migrating the `eth-json-rpc-provider` package into the core monorepo.
- Originally, the plan was to decrement the TypeScript version to align with the core monorepo, but following [this discussion](#27), we've opted to use TypeScript version v4.8.4, and preserve the compiler options in this package that aren't enabled in the core monorepo.

## References
- See MetaMask/core#1551
- Closes MetaMask/core#1683
- Closes MetaMask/core#1682 (is superseded by this PR)
MajorLift added a commit that referenced this issue Sep 29, 2023
…of migration (#28)

## Description
- This PR aligns the ESLint and Prettier packages and configs in this repo with the core monorepo.
- This is a preparation step for migrating the `eth-json-rpc-provider` package into the core monorepo.
- Originally, the plan was to decrement the TypeScript version to align with the core monorepo, but following [this discussion](MetaMask/eth-json-rpc-provider#27), we've opted to use TypeScript version v4.8.4, and preserve the compiler options in this package that aren't enabled in the core monorepo.

## References
- See #1551
- Closes #1683
- Closes #1682 (is superseded by this PR)
@MajorLift MajorLift reopened this Oct 5, 2023
MajorLift added a commit that referenced this issue Oct 11, 2023
…norepo

- added: `eslint-plugin-promise`
- removed: `eslint-plugin-node`

See: #1683
MajorLift added a commit that referenced this issue Oct 11, 2023
- added `eslint-import-resolver-typescript` v2.5.0 as dependency

See: #1683
MajorLift added a commit that referenced this issue Oct 11, 2023
…of migration (#28)

## Description
- This PR aligns the ESLint and Prettier packages and configs in this repo with the core monorepo.
- This is a preparation step for migrating the `eth-json-rpc-provider` package into the core monorepo.
- Originally, the plan was to decrement the TypeScript version to align with the core monorepo, but following [this discussion](MetaMask/eth-json-rpc-provider#27), we've opted to use TypeScript version v4.8.4, and preserve the compiler options in this package that aren't enabled in the core monorepo.

## References
- See #1551
- Closes #1683
- Closes #1682 (is superseded by this PR)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment