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

Upgrade dependencies #643

Closed
wants to merge 5 commits into from
Closed

Upgrade dependencies #643

wants to merge 5 commits into from

Conversation

philfuster
Copy link
Contributor

@philfuster philfuster commented Apr 16, 2024

This PR updates in mvom:

  • prettier
  • date-fns
  • npm-run-all -> npm-run-all2

npm-run-all2 is a maintenance fork of npm-run-all and is the community recommended replacement. The goal of the project is to have it all merged back mainstream when the owner of the npm-run-all repo gets back.

in the website:

  • clsx

@philfuster
Copy link
Contributor Author

@shawnmcknight I decided to handle this group of upgrades in one PR because they were the simplest. The remaining upgrades to the eslint suite of dependencies will be after/at the same time as removing the @storis dependencies and internalizing the configs there.

@philfuster philfuster marked this pull request as ready for review April 16, 2024 19:59
@philfuster
Copy link
Contributor Author

Dependencies that can’t be updated

  • eslint

eslint can't be upgraded to v9 because various plugins in use within mvom do not support eslint v9 yet. tracking issue - #646

  • @typescript-eslint/eslint-plugin and @typescript-eslint/parser

typescript-eslint/typescript-eslint#8211

  • eslint-plugin-jest

depends on @typescript-eslint plugins getting upgraded first.

@shawnmcknight
Copy link
Member

@philfuster I had merged the npm-run-all PR separately because I didn't realize this was out here. There are now merge conflicts here to address.

@philfuster
Copy link
Contributor Author

Going to merge the auto created PRs from renovate. No need for this.

@philfuster philfuster closed this Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants