Hadley Wickham comments

Results 2579 comments of


                                            Hadley Wickham

Bump required R version to R 4.0.0

Oh yeah, let me fix that too.

Tests fail unless I add source('setup.R') data to test file

I'm not sure what's going wrong with your code, but if you can make a simple reprex, I might be able to find time to look into it.

Tests fail unless I add source('setup.R') data to test file

I've closed this issue due to lack of requested reprex. If you still care about this bug, please open a new issue with a reprex.

Grouped boxplots that don't drop levels

I wonder if we should have `drop = FALSE` option to `position_dodge2()` to tell it to use the factor levels, rather than the actual positions? (Or maybe it's too late...

Grouped boxplots that don't drop levels

If you wonder why something so seemingly simple is so hard to fix, I'd suggest watching @karawoo's excellent rstudio::conf() talk: https://resources.rstudio.com/rstudio-conf-2019/box-plots-a-case-study-in-debugging-and-perseverance

Could the stack trace of `expect_no_*()` be reduced?

The mechanisms exist in rlang to trim the call stack; the challenge will be figuring out exactly what should form the `top` of the callstack and how do we get...

Could the stack trace of `expect_no_*()` be reduced?

I think I'm going to close this, just because when we've tried to automatically minimise the stack we've hit a case where we accidentally hide info that we do actually...

Failed to deserialize params.nodeId when attempting to click a button

Yeah, likely to be same problem as #405. I'll take a look when I'm next working on rvest.

FR: New `keep_empty = FALSE` argument to `list_c()`

Hmmmm, this doesn't feel quite right for purrr, but maybe it's the least worst place?

FR: New `keep_empty = FALSE` argument to `list_c()`

I think the way to implement this for both `list_c()` and `list_rbind()` is to replace any list elements that are identical to `NULL` with `NA`. Then the vctrs machinery can...