Lorenz Walthert

Results 538 comments of Lorenz Walthert
trafficstars

Ok finding out how `Rscript` gets ob your PATH is not a pre-commit issue. Does the same problem happen when you open a terminal with Apple‘s terminal app and try...

I think if you use VS Code and a reasonable recent version of macOS, then your default shell is zsh (and there should be no difference to opening a new...

Ok. I don't think I accounted for the situation where you don’t have a global R executable but instead only one in a conda environment. In fact, I think this...

How can you reliably detect if a directory uses a conda environment? And how to find the R executable?

> Ok, the availability of Rscript (and R in general) is a dependency of pre-commit itself and not your hook (https://pre-commit.com/#r) In pre-commit, there are [different levels of languages](https://github.com/pre-commit/pre-commit/blob/main/CONTRIBUTING.md#adding-support-for-a-new-hook-language), `r`...

I see the docs are wrong. However, in interest of style, I'd like to deprecate `--warn_only` and introduce `--warn-only` I think.

Strictly speaking I would not add the hook here, because it would be hosted, but we would add it to our template file.

https://github.com/google/yamlfmt requires go. I am not sure we want to add this requirement for our project. It would not affect anyone beyond the developers of {precommit} if we use it...

Hi @philipp-baumann. Unfortunately I don’t have any quick hints. The caches you mentioned before could be an issue, yes. Also, I had the same {pkgapi} warning before locally, and I...

Ok, after thinking a bit more about it, I think I have a clue. When I first set up the template for GitHub Actions, I had similar problems and I...