Usethis: browseURL() on RStudio Server

[jennybc]

The browseURL() usage in use_course() doesn't have the desired effect of opening the newly created directory on RStudio server. It attempts to open something like "8787/files/rematch2-master".

Reported by @mdsumner, who might care to elaborate here.

This is related to, but not the same as #129, in which browseURL() is being used to open an RStudio Project in, say, create_package().

[jennybc]

cc @hadley

[hadley]

Maybe should just suppress on RStudio server?

[jennybc]

Do you happen to know the official way to test if running on RStudio server?

[hadley]

rstudioapi::versionInfo()$mode

[jennybc]

Consultation with hosted apps confirms that yes, I should just suppress this entirely on RStudio server.

[jennybc]

@mdsumner Are you willing to install usethis from GitHub and confirm that use_course() behaves well on RStudio server? Specifically, it should not attempt to browse the newly created folder. The path should be clear from messages and the user will have to figure out how they want to proceed.

[mdsumner]

LGTM -use_course behaves well for me on RStudio Server ( also love how create_package now works to open the project :)

[jennybc]

Thanks!