-
Notifications
You must be signed in to change notification settings - Fork 2
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
Code cleanup #708
Code cleanup #708
Conversation
Thank you @jmafoster1 -- Yes, you can do a global cleanup. Hopefully it'll be smooth |
Global cleanup done. It only took an hour and a half, so not too bad really. I've tried the assistant on a few sites and everything seems to be working as before, but it's a monster of a PR to review! |
Thank you for cleaning up all the files. I went through all the files, tried the main features of the tools, and it is working as before. We really need automated tests 😨 |
Yes, very much so! Did you decide what to do about mocking in #655? I have some time this week before we start on the chat bot integration, so could look into that. |
Unfortunately, not yet, I hope I can give you an answer next week but it may need to wait until the week after |
prettier-plugin-sort-imports
plugin to order the imports consistentlyjscodeshift
plugins to convertimport {Thing} from "@mui/material"
imports toimport Thing from "@mui/material/Thing"
If you would like, I can run the new transforms globally to prevent commit noise later on, or we can gradually shift to the new formatting as files get updated.