Remove markdown and remotes from Suggests

[nanxstats]

This PR removes markdown and remotes from Suggests.

remotes was introduced in 04b36d0 for comparing data.table results to a previous dplyr-based version of simtrial in tests. Since the test has been removed in #135, remotes is not needed anymore.

markdown was introduced in 2118701 for unknown reasons. It is not used anywhere.

[jdblischak]

remotes was introduced in 04b36d0 for comparing data.table results to a previous dplyr-based version of simtrial in tests. Since the test has been removed in #135, remotes is not needed anymore.

Thanks for cleaning that up

markdown was introduced in 2118701 for unknown reasons. It is not used anywhere.

I suspect that was motivated by the change in knitr 1.34 to move markdown from Imports to Suggests. Though maybe not, since the commit was in May 2021, but 1.34 wasn't released until Sep 2021. Regardless, the vignettes were using the engine knitr::rmarkdown even back then, so I doubt that markdown was ever actually needed as a dependency

simtrial/vignettes/arbitraryhazard.Rmd

Line 6 in 2118701

%\VignetteEngine{knitr::rmarkdown}

[nanxstats]

I suspect that was motivated by the change in knitr 1.34 to move markdown from Imports to Suggests. Though maybe not, since the commit was in May 2021, but 1.34 wasn't released until Sep 2021. Regardless, the vignettes were using the engine knitr::rmarkdown even back then, so I doubt that markdown was ever actually needed as a dependency

Good reasoning - I don't think we ever used R Markdown v1 (via the markdown package) since R Markdown v2 (rmarkdown) was released like 10 years ago... so I'd say it's probably added by accident. 😅