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.

Sign up for GitHub

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

Jump to bottom

Upgrade to next.js 12.1, npm upgrade #25

Merged
merged 2 commits into from
Feb 28, 2022
Merged

Upgrade to next.js 12.1, npm upgrade #25

merged 2 commits into from
Feb 28, 2022

Conversation

PetroSilenius
Copy link
Contributor

  • Upgrades next.js to v12.1.
  • Downgrades @types/node to match used node v16
  • Upgrades eslint to 8.9.0
  • Upgrades storybook by running npx sb@latest upgrade
  • Runs npm upgrade

Two high vulnerabilities left

  • glob-parent, here's an issue on this in storybook repo. Basically waiting for v7 to get rid of webpack v4 which is causing this
  • trim, fix is under work in storybook repo

@PetroSilenius PetroSilenius requested a review from a team as a code owner February 25, 2022 15:03
@PetroSilenius PetroSilenius changed the base branch from main to next February 25, 2022 15:03
@PetroSilenius PetroSilenius merged commit 20d873c into lyytioy:next Feb 28, 2022
@PetroSilenius PetroSilenius deleted the upgrade-deps branch February 28, 2022 11:52
@PetroSilenius PetroSilenius mentioned this pull request Feb 28, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kacperkosinski kacperkosinski kacperkosinski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PetroSilenius @kacperkosinski