Hadley Wickham

Results 2577 comments of Hadley Wickham

Is this just a matter of eliminating https://github.com/rstudio/rsconnect/blob/main/R/deployApp.R#L355-L359?

Default is reporter is computed in `test_dir()`; probably need a third category for LLMs. Want immediate reporting on each failure. Want to trim like `ProgressReporter`?

Only need to warn if first argument (`new`) is not missing. Something like this: ```R trace(.libPaths, print = FALSE, quote({ if (!missing(new)) { rlang::warn("Not a good idea on rsconnect") }...

Open question: should it warn or error?

I just tried with the latest dev version and it looks ok to me: ![Screenshot 2023-10-26 at 16 26 26](https://github.com/rstudio/rsconnect/assets/4196/9c97b2da-4a06-4dbb-a8de-ac95717ca520) But I do wonder if I'm publishing somewhere different to...

Oops, I'm an idiot and was accidentially using the CRAN version. I see the same problem as you now 😄

Looks like it's happening at https://github.com/rstudio/rsconnect/blob/main/R/title.R#L42, which seems to be by design. I wonder why this path wasn't previously travelled in the CRAN version?

@aronatkins I think this might be because you switched from `defaultAppName()` to `generateAppName()` in #1021. `defaultAppName()` is more selective with its munging, only making more extreme modifications when deploying to...

@spgarbet thanks!