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 JS dependencies. #3161

Merged
merged 2 commits into from
Sep 21, 2021
Merged

Conversation

jgerigmeyer
Copy link
Contributor

@jgerigmeyer jgerigmeyer commented Sep 20, 2021

Upgrades JS dependencies, addressing security alerts.

Holding off on upgrading i18next for now; there are some breaking changes in v21 that don't (currently) play nicely with our auto-generated translation file. This may not be a problem in production, but will make it more difficult to track changes in translation strings going forward.

@jgerigmeyer jgerigmeyer changed the title 🐈 Upgrade deps, except i18next 🐈 Upgrade JS dependencies. Sep 21, 2021
@jgerigmeyer jgerigmeyer marked this pull request as ready for review September 21, 2021 15:42
@jgerigmeyer jgerigmeyer requested a review from a team as a code owner September 21, 2021 15:42
@jgerigmeyer
Copy link
Contributor Author

@davisagli I can't see the Dependabot security alerts (even on the OddBird fork with them enabled), but I think this should resolve the ones I'm aware of.

@davisagli davisagli merged commit 996bb9b into SFDO-Tooling:main Sep 21, 2021
@davisagli davisagli deleted the upgrade-deps branch September 21, 2021 16:03
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