Contributing to EpiNow2Source:
This outlines how to propose a change to EpiNow2.
You can fix typos, spelling mistakes, or grammatical errors in the documentation directly using the GitHub web interface, as long as the changes are made in the source file. This generally means you’ll need to edit roxygen2 comments in an
.R, not a
.Rd file. You can find the
.R file that generates the
.Rd by reading the comment in the first line of the
.Rd file in the
If you want to make a bigger change, it’s a good idea to first file an issue and make sure someone from the team agrees that it’s needed. If you’ve found a bug, please file an issue that illustrates the bug with a minimal reprex (this will also help you write a unit test, if needed).
Fork the package and clone onto your computer. If you haven’t done this before, we recommend using
usethis::create_from_github("epiforecasts/EpiNow2", fork = TRUE).
Install all development dependences with
devtools::install_dev_deps(), and then make sure the package passes R CMD check by running
devtools::check(). If R CMD check doesn’t pass cleanly, it’s a good idea to ask for help before continuing.
Create a Git branch for your pull request (PR). We recommend using
pre-committo check our changes match our package standards. This is optional but can be enabled using the following steps.
# if python is not installed on your system install.packages("reticulate") reticulate::install_miniconda() # install precommit if not already installed precommit::install_precommit() # set up precommit for use precommit::use_precommit()
Make your changes, commit to git, and then create a PR by running
usethis::pr_push(), and following the prompts in your browser. The title of your PR should briefly describe the change. The body of your PR should contain
For user-facing changes, add a bullet to the top of
NEWS.md(i.e. just below the first header). Follow the style described in https://style.tidyverse.org/news.html.
PRs are reviewed by the team before they are merged. The review process only begins after the continuous integration checks, which have to be manually triggered by a maintainer for first-time contributors, have passed.
The Github Actions checks currently take a while (about an hour), so it might be helpful to “watch” the repository and check your email for a notification when it’s all done.
Usually, all the review conversations occur under the PR. The reviewer merges the PR when every issue has been resolved. Please use the “Resolve conversation” functionality in the GitHub web interface to indicate when a specific issue has been adressed, responding with a commit pointing to the change made where applicable.
When a PR is ready to be merged, you may be asked to rebase on the
mainbranch. You can do this by checking out your branch and running
git rebase main. If it is successful, your commits will be placed on top of the commit history of
mainin preparation for a merge. A rebase might result in some merge conflicts. Make sure that they are resolved, then push your changes to your branch again (using the
--forceoption, that is,
git push -f, if required).
A number of issues can cause the Github checks to fail. It can be helpful to safeguard against them by doing the following:
Check that there are no linting issues by running
devtoools::check()to check for wider package issues like mismatching documentation, etc. (this currently requires a fair bit of time/computation).
(Optional) Turn on continuous integration with Github Actions on your forked repository.
We use testthat for unit tests. Contributions with test cases included are easier to accept.
Please note that the EpiNow2 project is released with a Contributor Code of Conduct. By contributing to this project you agree to abide by its terms.