Lorenz Walthert
Lorenz Walthert
Well, a simple workaround is to use an integer parameter (and supply 0, 1 only) and convert to boolean in your script.
I am not sure I’d do it that way. Of course you can pass flags, but mostly you want to use key value pairs and not just flags (not sure...
I realized it is more complicated because of permissions needed to execute this (and other problems I can't remember), so we solved the problem otherwise and I won't do a...
Thanks @nfultz. I know it's already taken for S4, but I think it would be still possible, since with S4, it's always *within* an expression as far as I know,...
Not sure I want to opt-in into an experimental feature, in particular due to > Once we're more certain the integration is solid we'll consider turning this on by default....
This affects the installation procedure of the hooks, which is not handled by {precommit} (or any repositories that implements hooks), but in the upstream framework, i.e. [here](https://github.com/pre-commit/pre-commit/blob/master/pre_commit/languages/r.py#L88). So testing this...
Locally, we could test it before by installing a development version of pre-commit from a branch (note pre-commit=upstream, {precommit} = R package).
@pat-s I'd welcome your comments.
Also: Lorenz Walthert think how to run all files through all hooks locally and when things pass, push.
@pat-s I just released a new hook version (i.e. `autoupdate()` will be sufficient): https://github.com/lorenzwalthert/precommit/releases/tag/v0.2.0.9001 where I incorporated some of the ideas from our discussion.