release() should call document()

[krlmlr]

before checking that all files are committed to Git. (Sorry if this has been suggested before.)

[jimhester]

It already implicitly calls document when it runs the package check, because the default is document = TRUE. It is true this is after the Git checks.

[hadley]

I think we'll probably totally rethink the release() function in the near future. I don't use it any more since it just seems to cause me more hassle than it saves.

[krlmlr]

I don't even remember how to manually upload a package to CRAN ;-) Would be interested to understand the hassle it's causing to you.

[hadley]

I don't know exactly - it's just some additional friction so I've basically switch to just using submit_cran(). I think it's probably just because my workflow has drifted a bit and so the questions aren't quite right.

[lock]

This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/