-
Notifications
You must be signed in to change notification settings - Fork 18
Optimizing FE further with dead code cleanup, documentation updates, removal of more convoluting dependencies #3025
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
Conversation
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.
Awesome, @alexjanousekGSA, thank you for all of this! 🎉
After reviewing it all and double checking several things, I only came across a couple of minor things to double check on/adjust. Everything else looks good to me!
Also, thank you for adding in the E2E test fixtures to help with cleanup!
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.
Thanks @alexjanousekGSA for your efforts here. Looks 10000% better already. I left a few comments, mostly suggestions or questions
ccostino
left a comment
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.
Awesome work, thanks again @alexjanousekGSA!
This PR has a few minor issues addressed:
Larger issue:
Upgraded Node.js from 22.3.0 to 24.10.0 (upcoming LTS). Node 24 will enter Long-Term Support in October 2025 and receive support until April 2028.
Below is the current node release schedule.