Michael Chirico

Results 324 comments of Michael Chirico

Another thought: It might help beginners to see `drive_update()`/`drive_put()` mentioned in this output too? https://github.com/tidyverse/googledrive/blob/74a69a3a1fd66e930802ce6e461538c9e15f7c86/R/utils-paths.R#L59-L61

Since some of those `iter=` changes made it in already in #252, WDYT about merging this PR without that change, and I'll file a follow-up that reverts all `iter.max=` usage...

`seq(along=)` _might_ be an older practice from before `seq_along()` was added as a function? {survival} is _far_ from the only package using that style, here's 14K hits for the same...

PS there wound up being a lot of `.Rout.save` files that don't have any substantive update (since I just copied over _every_ `.Rout` file from `R CMD check`), would you...

> I'm still grappling with how best to use git, so have patience while I work that out, in particular how to pull things down to my test bed and...

I did see this in the original commit for the cardinal_ version: https://github.com/boostorg/math/commit/caf1883788da6fcbfd293e805806b312e595963a (but I also can't make heads or tails of it). Such a PR would be great! I've...

yes I had this in mind as going into yoda_test_linter()

This covers a TODO as well: https://github.com/r-lib/lintr/blob/72eeefd87596ea7fa72ee1a8b14bcb82d322c381/tests/testthat/test-yoda_test_linter.R#L73-L75

Related issue (please LMK if this is more appropriate as an independent issue), continued from https://github.com/r-lib/pkgload/issues/239. I would expect `roxygenize()` to work even in the absence of a NAMESPACE file...

Maybe there could be a new roclet `initiate_namespace`? That only handles the `@import`/`@importFrom` part of the current `namespace` roclet? Not sure if that actually gets around the problem in any...