Lorenz Walthert comments

Results 536 comments of


                                            Lorenz Walthert

trafficstars

Add newline after multiline function declaration

> For a propper implementation, we'd need #357 because we want to add a line break dependent on whether a token is multi-line, which we cannot do if the multi-line...

Wrap long lines (follow up)

Status of this PR: - worked at some point but not for all cases I experimented with. - probably there are merge conflicts by now, has to be rebased on...

Default to styling all supported file types in `style_dir()` and `style_pkg()`?

Yes. Also thought about it. Did not want to do that in the very beginning though, when formatting of non-R was experimental.

Default to styling all supported file types in `style_dir()` and `style_pkg()`?

This issue is also included in #769, although questionable if breaking or not. PR welcome yes.

Use testthat::teardown() for propper clean-up of caching related issues

Superseded by #783.

convert `%>%` into `|>`?

It would be optional. And it's aimed at being very simlar with release 2.0: https://www.tidyverse.org/blog/2020/11/magrittr-2-0-is-here/

Ignore {knitr} chunks that use `Rcpp` engine?

Thanks. The quick fix I beliebe is just not to use `r` at the beginning, as the first unnamed argument of a code chunk header seems to get matched with...

Ignore {knitr} chunks that use `Rcpp` engine?

Also note other issues that have been filed with similar requests #832, #312.

Ignore {knitr} chunks that use `Rcpp` engine?

Thanks, I'll file an issue in {knitr} and ask about exporting this function. Really strange these emails you get...

Ignore {knitr} chunks that use `Rcpp` engine?

strange. Because I never get any e-mail...