Andy Teucher

Should be added to internal function `add_readme()`

Bundling it is a great idea, and it's open license so no problem. As for a Word template, I think it's meant to be a web font, not necessarily for...

As of usethis 1.5.0 there are many new goodies for using usethis in other packages: https://www.tidyverse.org/articles/2019/04/usethis-1.5.0/

This is a great catch, thanks @henry-ngo. We are planning on changing the default branch for all the repos that we can, so this would bite eventually bite us (and...

For the code, we should get it dynamically using the GitHub API: library(gh) gh("/repos/bcgov/bcgovr")$default_branch #> [1] "master" And in the `CONTRIBUTING.md` file, just be generic and refer to it as...

This is relevant: https://github.com/r-lib/usethis/issues/1156

@henry-ngo did you actually run into this, or just notice it in the source code? Remembering back, this whole [`tryCatch` block](https://github.com/bcgov/bcgovr/blob/affe8ce20286a6e53cb11be0a6e0aec1027eb7cd/R/use.R#L203-L215) was just there to [work around](https://github.com/bcgov/bcgovr/commit/60f8bc43e6b85853153c89a76c2a98a2e8edb32c) a bug in...

Ok, thanks. Perhaps we still do need that catch then. Just to make sure I understand, you hadn't created an empty repo on GitHub beforehand, right?

Yes I think so... and I think maybe we should remove that tryCatch anyway, the error is between user and `usethis`, maybe we don't want to insert a fragile fix...

Oh yes, it probably is! It doesn't make the vignettes available locally (i.e., with `browseVignettes()`) but solves the problem in a different way...