-
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
Naming for Util #328
Comments
Discussed at 8/25/2015 meeting and the consensus was that this doesn't bother us too much but if someone wants to make them consistent, it would make life slightly easier. Either Util or Utils is fine. |
It would make life slightly easier if we picked either 'Utils' or 'Util' and used it consistently. I don't feel a need to expand to "Utilities", since it's such a common abbreviation. |
What about renaming DOT/Util to DOT/DotUtil so that you can use it together with other Utils? etc. |
See phetsims/dot#4 |
Ah right, thanks for reminding me about the dot issue. I'll unassign this. |
What is the status of this issue, seems to have gone a bit stale? |
The consensus (summarized in #328 (comment)) is that this is a minor inconvenience and a major pain to change, so someone will work on it if/when they feel the urge. Imo it's not tragic if it's never addressed; we have much bigger fish to fry. |
considering comment above, closing |
We have a few different conventions for our Util classes.
Util.js found in dot, mobius, scenery, optics-lab, trig-tour

Utils.js found in shred, build-an-atom
Sim-specific util files as noted below, such as ChipperStringUtils.js
It would be nice to standardize on a naming for our utility files. Also, we decided long ago to use full names instead of abbreviations for many things--util seems to be one of the last holdouts. Let's discuss where to go from here. Also note having many files with the same name causes import conflicts and difficulty when navigating by filename.
Discovered this issue during phetsims/chipper#116
The text was updated successfully, but these errors were encountered: