-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix Dependabot #124 cross-spawn
; Upgrade tiptap
, css-loader@7
#1303
Conversation
Also upgrades a host of Babel packages to the latest patch
cross-spawn
, upgrade CSS-Loader@7cross-spawn
, upgrade css-loader@7
@@ -3,7 +3,7 @@ import PropTypes from 'prop-types'; | |||
|
|||
import classNames from 'classnames'; | |||
|
|||
import styles from './CheckboxButton.module.scss'; | |||
import * as styles from './CheckboxButton.module.scss'; |
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.
these were the breaking changes for css-loader https://github.com/webpack-contrib/css-loader/releases/tag/v7.0.0
"@tiptap/pm": "^2.7.4", | ||
"@tiptap/react": "^2.7.4", | ||
"@tiptap/suggestion": "^2.7.4", | ||
"@tiptap/core": "^2.10.3", |
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.
Basically I clicked through each component in the app and confirmed they were loading with no iframe bundle
errors, ran lint
cross-spawn
, upgrade css-loader@7
cross-spawn
, upgrade tiptap
, css-loader@7
cross-spawn
, upgrade tiptap
, css-loader@7
cross-spawn
; Upgrade tiptap
, css-loader@7
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.
lgtm
Started fixing https://github.com/user-interviews/ui-design-system/security/dependabot/124
But kept pushing the rest of our dependencies and got everything to work except I didn't try the latest eslint given it is usually a huge pain in the butt.
Major upgrade of
cssloader
@ 7.xUpgrades minor/patch:
Tested each component renders in Storybook: