Support Quarto articles

[olivroy]

Now that upcoming pkgdown version supports them!

fixes #1997

With the release of pkgdown 2.1, I think it is worth adding to usethis!

I set overwrite = FALSE to setting the VignetteBuilder field. Not sure how having quarto and rmarkdown vignettes will work. I can do more testing after review!

Would need the following too in Rbuildignore https://github.com/r-lib/pkgdown/pull/2656/files

^vignettes/\.quarto$

[jonthegeek]

This will be super helpful! I suggested a few changes, but this is well on its way! You'll also need to resolve a merge conflict in the NEWS.md (I couldn't fix that piece).

[jonthegeek]

Suggested change

	#' @param type One of `"quarto"` or `"rmarkdown"`
	#' @param builder One of `"quarto"` or `"rmarkdown"`

type is a little over-generic and could collide with another param. Let's use the more-specific builder (to match "VignetteBuilder"). If you go with this change, it needs to be changed throughout.

[olivroy]

agreed that type may not be ideal. If we were to go with builder, the arguments should be updated to "knitr" instead of "rmarkdown"?

Maybe ext = c("qmd", "rmd") would work too.

Maybe there should be an option

options(usethis.vignette = "quarto") to avoid specifying it everytime?

I can imagine myself deciding to stick with a single type, and not have to specify it everytime.

[jonthegeek]

I like ext, and an option would be great!

[olivroy]

Thanks so much for your helpful comments! I will get this in a better shape

[jennybc]

Thanks to both @olivroy and @jonthegeek! I won't be able to complete thinking about this and merge this today, but I will definitely do so soon. And we're clearly very close!

[coatless]

@jennybc not sure when you'll have cycles next; but, if you could hint at any changes required to take the Quarto vignette action across the finish line, I'm happy to help.

[coatless]

Gentle bump on adding Quarto vignettes into use_vignette() via builder = "quarto".

[jennybc]

I will come back to this soon, as I need to release usethis for CRAN reasons.

[jennybc]

@olivroy Thanks again for getting this ball rolling. Returning to this task, I just found it easier to tackle it myself de novo, in #2085. I will make sure to give you a shout out in the NEWS.