-
Notifications
You must be signed in to change notification settings - Fork 73
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
Memorize which files were not changed since last styling #320
Comments
@krlmlr I think this is worth exploring, it goes into the same direction as parallelization. There are quite a few design questions involved with this, so I'd like to hear your opinion on this. Should we use a |
A directory or a separate file would be better, we don't want to mix config data and hash data. |
Also, let's compare our ideas with other source code formatters: black uses a non-portable file stored in some cache, the path to which is system dependent: psf/black#136. |
We can use the R.cache package to make sure we cache things in the right location OS-independently. Then follow: psf/black#109 (comment). We can store version of styler as well as the hash of the file content (requires reading them, as opposed to use file size and modification time), then.
Benefits of this approach are (credits to psf/black#109 (comment)):
|
Could significantly speedup
style_file()
andstyle_dir()
when used often as well as pre-commit hooks.Needs new infrastructure.styler
folder in root directory.The text was updated successfully, but these errors were encountered: