build_home(): reduced priority for "index.Rmd" in pkgdown v1.4

[mrustl]

While in pkgdown v1.3 the priority for building the home index.html was as follows:

pkgdown/R/build-home-index.R

Line 9 in 42971af

c("index.Rmd", "README.Rmd", "index.md", "README.md")

It was changed with pkgdown v1.4 to:

pkgdown/R/build-home-index.R

Line 8 in 383c7ee

c("index.md", "README.md", "index.Rmd", "README.Rmd")

In order to be backward compatible, it would be great if the priority could be changed back to the one used in pkgdown v1.3 or as follows:

c("index.md", "index.Rmd", "README.md", "README.Rmd")

This would be really helpful as our > 40 company open-source R package repositories (https://kwb-r.github.io/status), which all use pkgdown, rely on the fact that index.Rmd is prioritised higher than README.md

[hadley]

pkgdown no longer renders .Rmd to .md, so I don't think changing the order of that vector will help you. Instead, I think you'll need to add a rmarkdown::render() in your deployment process.

[mrustl]

@hadley thanks for explaining this. This was not obvious for me as the pkgdown v1.4 documentation just mentions:

build_home() no longer re-builds README.Rmd
https://pkgdown.r-lib.org/news/#new-features

In addition listing the ".Rmd" files in build_home_index() is counterintuitive:

pkgdown/R/build-home-index.R

Line 8 in 383c7ee

c("index.md", "README.md", "index.Rmd", "README.Rmd")

For being more explicit on this I propose #1140