-
Notifications
You must be signed in to change notification settings - Fork 14
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
Useful script to convert all .js files to .ts in a repo #1120
Comments
Remember if renaming a file, it should be done with |
Agreed! I have 1364 compiler errors to get through before I'm done, but I thought it would go fast for this sim. As for using
|
I updated the docs to match my current sentiment, can you please review and advise? |
Just to note here for the record. I did go "all in" for ratio and proportion with that script, and in general it was not totally ideal. I would likely do it again but wanted to note the trade offs. Because it ended up taking a couple of days to work through the whole repo, I am committing to a branch so that master doesn't have compile errors. That said, it has been really nice to have a global number of all compile errors to track my progress. First day I went from 1300 -> 1100, then over the next 2 days I got to 500. I think I would use this script again if it was 8am, and I was planning on converting a repo as my whole day today, because I think it is really tiresome to convert file extensions manually. Anywho, the RaP issue I'm tracking this in is phetsims/ratio-and-proportion#404, and note that it is on a branch (for now. . .). |
Thanks, sounds interesting and I'll consider it as a possibility for myself next time! |
While working on phetsims/ratio-and-proportion#404, I found this to be useful:
@samreid, do you think this could be part of the documentation on converting repos to typescript?
The text was updated successfully, but these errors were encountered: