-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
chore: Replace usage of moment in cli / server code #9069
Conversation
Thanks for taking the time to open a PR!
|
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
I'm guessing the cli types are there because of |
Yeah, I am expecting this to be removed likely from the driver, but think this will probably be better slated for 7.0. Maybe we could add a deprecated warning soon though. |
I'm in favor of this change, users can import |
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.
Didn't manually test but knowing the two libraries have the same API and tests pass, LGTM
The |
User facing changelog
N/A - this is not user facing
Additional Details
We'll be moving forward with removing moment.js from the driver and cli types in the 7.0 breaking change.