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

Update dependencies & remove unused code #764

Merged
merged 5 commits into from
Jan 23, 2025
Merged

Conversation

kmcginnes
Copy link
Collaborator

@kmcginnes kmcginnes commented Jan 23, 2025

Description

This set of updates should resolve #760 while also updating all dependencies.

I performed a "safe" updated with pnpm -r update and verified the app still works with a smoke test.

Also included is the removal of some code that was not called and two dependencies that are no longer used (and marked DEPRECATED in NPM).

  • Removed some unused components in the tabular set of code
  • Removed dependency on react-beautiful-dnd and @types/history
  • Updated all dependencies to compatible latest minor and patch versions

Validation

Related Issues

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0
    license.
  • I have run pnpm checks to ensure code compiles and meets standards.
  • I have run pnpm test to check if all tests are passing.
  • I have covered new added functionality with unit tests if necessary.
  • I have added an entry in the Changelog.md.

@kmcginnes kmcginnes marked this pull request as ready for review January 23, 2025 21:07
@kmcginnes kmcginnes merged commit 90cc8c0 into aws:main Jan 23, 2025
1 check passed
@kmcginnes kmcginnes deleted the update-deps branch January 23, 2025 22:53
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