-
Notifications
You must be signed in to change notification settings - Fork 6
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
jest routing npm package update #1625
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Is there anything we shoudl look out for with such a big version jump on two of the packages?
"jest-axe": "5.0.1", | ||
"jest-environment-jsdom": "29.5.0", | ||
"jest": "29.7.0", | ||
"jest-axe": "9.0.0", |
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.
This is a big jump. Are there any concerns?
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.
I had some concerns regarding the WCAG standard being used and our tests breaking but no, it was seamless. We should update this though because this updates the min node version to 16.
@@ -76,7 +76,7 @@ | |||
}, | |||
"devDependencies": { | |||
"@babel/core": "7.14.6", | |||
"@jest/core": "27.3.1", | |||
"@jest/core": "29.7.0", |
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.
Jmping up two major versions. Any concerns?
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.
No, there are bug fixes and some improvements but no breaking changes.
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.
Sounds good to me!
This PR does the following:
How has this been tested?
Locally.
Accessibility concerns or updates
Checklist:
Front End Review: