-
Notifications
You must be signed in to change notification settings - Fork 3
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
[MR-886] Upgrade react-scripts to v5 #914
Merged
+6,260
−7,043
Merged
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
285d7f6
Bump react-scripts from 4.0.3 to 5.0.0
dependabot[bot] bb1985a
Upgrade uswds even though its out of sync with react-uswds to get the…
haworku 3316e54
Remove tilde from sass imports, since its deprecated, and use full fi…
haworku 5106595
Upgrade storybook react dependencies, remove pinned deps from react-s…
haworku 85cdbe3
Remove unused dependenct react-aria-modal
haworku 864852e
Fix up more issues related to scss compiling
haworku 39ce622
Revert "Upgrade storybook react dependencies, remove pinned deps from…
haworku 2cb577e
Fixing uswds asset path reduces errors from 227 to 162
haworku c052434
Revert "Revert "Upgrade storybook react dependencies, remove pinned d…
haworku 90be8a4
Address all compile errors, on to warnings
haworku 9f941b3
Upgrade sass
haworku 7635509
Manually resolve react router deps
haworku 79e9da8
Remove node modules and regenerate yarn.lock
haworku cca0caa
Upgrade to storybook 6 and add cra5 builder
haworku 0aab907
setup router compatibility package
haworku ab2dbcb
Upgrade to next storybook prerelase, which pre-bundles @storybook/rou…
haworku 31cd211
Temp commit - comment out all reference to history to see if we can …
haworku b8b2f85
Address Typescript warnings in ChangeHistory.test.tsx due to incorrec…
haworku 542aa95
Apparently we need to import Buffer now to use it in app-web - fix co…
haworku ea800da
Revert "setup router compatibility package"
haworku 77b80b4
Add serverless-webpack to uploads to address CORS error from webpack …
haworku eac7aae
Remove unnecessary history dep
haworku a0946d6
Merge remote-tracking branch 'origin/main' into hw-upgrade-react-scripts
haworku ebf9560
app-web tests passing
haworku c2b10e9
Add the other webpack dep from app-api to uploads, see if this gets i…
haworku 64b021d
Upgrade ts-jest and jest to address app-api errors
haworku e2369b7
Sync up versions of webpack deps in app-api and uploads
haworku 62c9ca2
Bump packages
mojotalantikite 3f095a5
use /scss paths for uswds deps
haworku 4897bc5
Bump mem limit on CI tests
mojotalantikite db83bc9
Merge branch 'hw-upgrade-react-scripts' of github.com:CMSgov/managed-…
haworku 4989462
Adjust module.scss paths to styles deps
haworku 242d3b5
Merge remote-tracking branch 'origin/main' into hw-upgrade-react-scripts
haworku adf1882
Add layer to only graphql
mojotalantikite f4cd48a
Debug what's setting up the prisma conn
mojotalantikite 574b0b9
Include more prisma things
mojotalantikite 0ae61fd
Add more prisma generate
mojotalantikite 61fd4dc
prisma is not global in CI
mojotalantikite c3153cf
Have prisma generate from lerna for CI
mojotalantikite 5a738dc
Add webpack5 syntax. Add prisma via webpack.
mojotalantikite 6a694d1
Try to force only lambda layer
mojotalantikite c6aa4f5
Ignore all prisma in webpack. All of it.
mojotalantikite f4e3a11
Try the monorepo fix for externals
mojotalantikite 7395f41
Add the schema file back
mojotalantikite d818bad
Didn't take. add again
mojotalantikite b41dd25
Bump ts-loader
mojotalantikite db2b772
Merge branch 'main' into hw-upgrade-react-scripts
mojotalantikite bac107c
Prisma should be layer only now
mojotalantikite 428cf21
Have serverless force exclude prisma. In a layer.
mojotalantikite 5428080
Fix cleaning
mojotalantikite 953a964
Remove debug
mojotalantikite e6004db
Merge branch 'main' into hw-upgrade-react-scripts
mojotalantikite b5fdf07
Merge branch 'main' into hw-upgrade-react-scripts
mojotalantikite a855986
Sync resolutions. Make sure types are in devdep
mojotalantikite caa137d
Merge remote-tracking branch 'origin/main' into hw-upgrade-react-scripts
haworku 6b79287
app-web lint passing
haworku 206616d
app-api linting passing
haworku dd097a5
Fix app-web ReviewSubmit tests again
haworku 7766052
Comment out problematic cy.pa11y calls to unblock this PR merge.
haworku File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,16 @@ | ||
module.exports = { | ||
stories: ['../src/**/*.stories.@(ts|tsx)'], | ||
addons: [ | ||
'@storybook/addon-a11y', | ||
'@storybook/addon-links', | ||
'@storybook/addon-essentials', | ||
'@storybook/preset-create-react-app', | ||
], | ||
typescript: { | ||
check: false, | ||
checkOptions: {}, | ||
reactDocgen: 'react-docgen-typescript', | ||
reactDocgenTypescriptOptions: { | ||
shouldExtractLiteralValuesFromEnum: true, | ||
propFilter: (prop) => | ||
prop.parent ? !/node_modules/.test(prop.parent.fileName) : true, | ||
}, | ||
}, | ||
}; | ||
stories: ['../src/**/*.stories.@(ts|tsx)'], | ||
addons: ['@storybook/addon-a11y', '@storybook/addon-links', '@storybook/addon-essentials', '@storybook/preset-create-react-app'], | ||
typescript: { | ||
check: false, | ||
checkOptions: {}, | ||
reactDocgen: 'react-docgen-typescript', | ||
reactDocgenTypescriptOptions: { | ||
shouldExtractLiteralValuesFromEnum: true, | ||
propFilter: prop => prop.parent ? !/node_modules/.test(prop.parent.fileName) : true | ||
} | ||
}, | ||
core: { | ||
builder: 'webpack5' | ||
} | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
why is this change in the react-scripts PR?
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.
Looks like this came in with adf1882. @mojotalantikite @macrael does that commit look alright to yall just sanity check because we had a bit of funky stuff with app-web merge at least want to be sure.
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.
So yeah, with the webpack5 changes there was a change in behavior that stopped prisma from working again. Basically it was copying in parts of the prisma client, but not any of the engine things that it needs.
In order to get it working again I added a single layer that has the prisma packages and forced webpack to not pack up prisma. That way the resolutions for prisma go to the lambda layer, which have the appropriate query engine builds.