Replace conditional S3 method registration with NAMESPACE directive

[MichaelChirico]

tinytable/R/zzz.R

Lines 1 to 5 in 2260238

	.onLoad <- function(libname, pkgname) {
	if(requireNamespace("knitr", quietly = TRUE)) {
	registerS3method("knit_print", "tinytable", knit_print.tinytable, envir = asNamespace("knitr"))
	}
	}

Was searching around a bit after discovering the very nice approach in Rdatatable/data.table#6589 and found {tinytable}. Because {tinytable} depends on R>=4.1.0, this can be replaced by S3method(knitr::knit_print, tinytable).

I only didn't file the PR because I am not sure the right way to do so with {roxygen2} (do we need @rawNamespace?), and don't quite have time to dig deeper at the moment.

[vincentarelbundock]

Thanks for raising this issue. This looks interesting. I don't have time to figure this out now, but I'm definitely interested and will try to take a look when possible.

[vincentarelbundock]

Thanks again. This appears to work:

#' @rawNamespace S3method(knitr::knit_print, tinytable)

8fea197

[MichaelChirico]

Nice! FWIW we found roxygen2 has slightly cleaner support:

r-lib/lintr@c3d2935