Lorenz Walthert
Lorenz Walthert
**Edit: this is based on an old wrong diff:** Just from glancing over the first 10% of our test files, I found substantial differences like (in chronological order) * No...
@lionel- `inst/air/overwrite-in-test-files-with-out-files.R` does only the re-naming, as the name suggests. I called _air_ before I called the re-naming script, as air formatted in place the `-in.R` files, whereas our test...
ok @lionel- my bad, something went wrong. I think `air` failed since some files were not formatted due to parsing errors: ``` lorenz@lorenzs-macbook-air-2 styler % air format tests/testthat/**/*-in.R thread 'main'...
Sounds good. I am glad running air on our test suit helps polishing your release 😊
I think we can conclude this experiment. Cheers.
Thanks for letting me know gem know @averissimo. Seems like my upstream fork (of pre-commit, the python package) is not enough... Do you think there is a generic solution for...
Thanks for looking into it. So you recommend setting the env variable when installing the hooks with `$ pre-commit install`? Would that then install all packages from source and from...
> Edit: I just realized that it helped in a synthetic test—when invoked by hand. I cannot figure out how to pass absolute path to the repo via pre-commit's args....
Sorry @TymekDev I took so long to answer. I can reproduce your problem, thanks for the detailed report. In case this was not evident already, here's how I'd adapt the...
I consider this out of scope, mainly because other people already looked into this. E.g. you could use {styler} with the [jupyter lab formatter](https://jupyterlab-code-formatter.readthedocs.io/en/latest/installation.html#installation-step-2-installing-a-supported-code-formatter). ~Maybe we should add a note...