Michael Chirico

Results 322 comments of Michael Chirico

I also get the e-mails btw On Thu, Mar 10, 2022 at 3:18 PM Lorenz Walthert ***@***.***> wrote: > Thanks, I'll file an issue in {knitr} and ask about exporting...

Sure, I will give it a go. It will take a while to get to it.

It also seems like `styler` insists `switch()` be split across multiple lines, even when it fits cleanly in just one, e.g.: ``` styler::style_text("switch(tolower(suffix), k = 1, m = 2, g...

Thanks, that's unfortunate, it really is a huge waste of space for cases when it can fit inline. I'll file upstream -- the one-line examples in the guide all use...

That's https://github.com/tidyverse/style/issues/181

doesn't this apply as a reason not to always restyle named arguments? > You may also place several arguments on the same line if they are closely related to each...

if you're doing a lot of in-place overwrites / it's a bottleneck, maybe worth considering `data.table` as a backend. Happy to advise/author some work towards that.

> looks like a major refactoring That's likely true :) And ah, I didn't read the above closely enough. I thought https://github.com/tidyverse/tibble/pull/1469 was a working branch shifting to data.frames on...

That makes sense, and I did file https://github.com/tidyverse/style/issues/185 simultaneously with this. I filed this instead of just waiting for upstream because of the apparent inconsistency. Conversely, would you also say...