-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Decide on package development workflow and release policies #535
Comments
It uses auto deployment so it's updating the dev version on develop and main (not that it has a develop version). If we made it have a non dev version it would make them release docs |
Note sure I can follow entirely. That means the semantic version number controls what gets released to the docs and that number is controlled by auto deployment? |
stick with the current workflow and prevent the docs being deployed on develop I think. Use auto deployment in the pgkdown yaml and dev package versions that the CRAN docs are at the root of the address and the develop docs (i.e from main) are at /dev/ |
Summary of what we're planning to do:
Question: what do we do when we have like 50 commits on |
I think the solution there is to make a release branch and cherry pick the commits that we want from the development version into it and then release to CRAN from there. |
We should decide on a coherent policy for how we develop the package. Relevant questions are: where does the development version sit and how should we manage the docs at https://epiforecasts.io/scoringutils?
Status quo:
Our Readme says:
install.packages("scoringutils", repos = "https://epiforecasts.r-universe.dev")
main
branch:remotes::install_github("epiforecasts/scoringutils", dependencies = TRUE)
.In reality:
main
currently equal to the stable version on CRANdevelop
branchmain
ordevelop
(even before merging intodevelop
)This seems pretty bad and we should urgently fix that.
Possible solutions
Option A:
main
Option B:
main
. Pretty much every change tomain
should correspond to a CRAN releasemain
develop
. Readme should point to thatOption C? I don't have a very good grasp of what best practices should look like. But it seems to me like we should make sure that the docs at https://epiforecats.io/scoringutils correspond to the version on CRAN, otherwise things can quickly get confusing for users.
The text was updated successfully, but these errors were encountered: