Skip to contents

Contributing to AQEval

Source: .github/CONTRIBUTING.md

This outlines how to propose a change to AQEval.

The text is based on the template provided by the folks who developed the tidyverse packages.

Please see the tidy contributing guide for further details regarding contributions to tidyverse packages, and good practices for those contributing to open software, data sets and archives more generally.

Fixing typos

You can fix typos, spelling mistakes, or grammatical errors in the documentation directly using the GitHub web interface or as a pull request (see below).

Documentation is generated using roxygen2, so you will need to edit comments in the roxygen2 comments section
in the related .R relevant script, rather than as discrete a .Rd file. (That .Rd file can then be built from the .R using devtools::document())

Bigger changes and code changes

If you want to make a bigger change, it’s a good idea to first file an issue and make sure someone from the package team agrees that it’s needed and don’t hinder established user work-flows.

But, please note, some major changes in the main functions may not be possible and some many take time to test and implement.

Likewise, if you’ve found a bug, please file an issue. We would like you to illustrate the bug with a minimal reprex example and we may ask you to do that if we have problems replicated any non-reprex bug reports, but we would appreciate a ‘head-up’ regardless of the format if you think anything is not behaving…

Pull request process

  • Fork the package and clone onto your computer. If you haven’t done this before, tidyverse folks recommend using usethis::create_from_github("karlropkins/AQEval", fork = TRUE).

  • Install all development dependencies 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 get in contact before continuing.

  • Create a Git branch for your pull request (PR). Here the tidyverse recommendation is to use usethis::pr_init("brief-description-of-change").

  • Make your changes, commit to git, and then create a PR (tidyverse recommendation usethis::pr_push() then follow the prompts in your browser). Ideally, the title of your PR should briefly describe the change, and the body of your PR should contain Fixes #issue-number.

  • For user-facing changes, please include a suggestion for a NEWS.Rmd bullet.

Code style

  • Please feel free to follow tidyverse style guide regarding coding practices and format, but if it works and we can understand it, we are not going to waste anyone’s time enforcing specific coding practices.

  • We use roxygen2, with Markdown syntax, for documentation. This we require you to also do.

  • We currently do NOT use testthat for unit tests. We would appreciate worked examples demonstrating a code modification but, please, no testthat.

Code of Conduct

Please note that the AQEval project is released with a Contributor Code of Conduct. By contributing to this project you agree to abide by its terms.