Review exported non-internal functions

[hadley]

Just to make sure we're not advertising anything that we'd no longer recommend.

Functions to think about:

• release()
• reload()
• bash()
• wd()
• dev_mode()

[jennybc]

Thinking about #2489 made me realize that dev_mode() is arguably superseded at this point. Which might be good to formally declare.

OTOH I am still a keen user of release(). I like all those questions.

[hadley]

@jennybc are you still a release user?

[hadley]

Other functions to consider:

• build_vignettes() + clean_vignettes()
• reload()
• missing_s3()
• show_news()

Maybe create()?

[jennybc]

No, I don't really use release() anymore.

[hadley]

All in one place with recommendations:

• bash() -> deprecate
• build_vignettes() + clean_vignettes() -> Discourage use of build_vignettes() #2488
• create() -> deprecate in favour of usethis::create_package()
• dev_mode() -> deprecate
• missing_s3() -> deprecate (now done by roxygen2)
• release() -> Steer people away from release() #2500
• reload() -> deprecate
• show_news() -> deprecate
• wd() -> deprecate

[jennybc]

Yes, I am on board with the latest summary of functions to discourage through the strongest means available.