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

Revert Dependencies #574

Merged
merged 1 commit into from
Aug 20, 2024
Merged

Revert Dependencies #574

merged 1 commit into from
Aug 20, 2024

Conversation

sbSteveK
Copy link
Contributor

@sbSteveK sbSteveK commented Aug 20, 2024

Dependencies were updated using npm-check-updates command in this PR: #554 but the updates were determined against Node 22. Our current minimum supported version is Node 14 and some dependencies were updated that are triggering minimum Node version warnings. Reverting dependency versions and updating the specific dependencies that trigger dependabot alerts:
axios v1.6.8 -> v1.7.4
tar v6.2.0 -> v6.2.1
@aws-sdk/util-utf8-browser v3.109.0 -> v3.259.0

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sbSteveK sbSteveK changed the title Fix Dependencies Revert Dependencies Aug 20, 2024
Copy link
Contributor

@xiazhvera xiazhvera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR looks good to me, please remember to add the steps to properly update the js dependency.

@sbSteveK sbSteveK merged commit d2f7e6b into main Aug 20, 2024
30 checks passed
@sbSteveK sbSteveK deleted the fix-dependencies branch August 20, 2024 21:00
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