-
Notifications
You must be signed in to change notification settings - Fork 19
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
Turn off modification times by default #43
Comments
I agree. Also because the main use case for the normalizer IMO is to avoid unnecessary diffs when the UFOs are under git version control. And the modification timestamps change by definition all the time. |
I'm not opposed to that change, but I'd like to hear other people's opinion first. I understand the motivation — I added the |
Isn't that file called layerinfo.plist? ;)
|
No because layerinfo is part of the ufo source, and should not be gitignored. We can store them next to the ufo, and name the dotfile with the same name as the ufo it is normalising |
Or maybe in the data folder would be better (ufo3 only) |
I like the data folder idea. |
I regularly trip over this because I forget using the
-m
parameter, and using it won't erase the mod times already written to disk. Since a normalizer should normalize and not jot down mod times, I petition to turn it off by default.The text was updated successfully, but these errors were encountered: