This contributing guide has been derived from the tidyverse boilerplate (see their high-level contributing guide). If you have any questions about contributing, please don’t hesitate to reach out. We appreciate every contribution. We suggest first reading the Getting started with tidytags vignette.
Contributor Code of Conduct
Please note that this package is released with a Contributor Code of Conduct. By contributing to this project, you agree to abide by its terms.
Non-technical contributions to tidytags
Feel free to report issues:
- Questions are for seeking clarification or more information. Both question askers and question answerers are welcome contributors!
- Bug reports are for unplanned malfunctions. If you have found a bug, follow the issue template to create a minimal reprex.
- Enhancement requests are for ideas and new features.
Technical contributions to tidytags
If you would like to contribute to the tidytags code base, follow the process below:
- PR process
- Code of conduct
This explains how to propose a change to tidytags via a pull request using Git and GitHub.
For more general info about contributing to tidytags, see the Resources at the end of this document.
To test the tidytags package, you can use an openly shared TAGS tracker that has been collecting tweets associated with the AECT 2019 since September 30, 2019. This is the same TAGS tracker used in the Using tidytags with a conference hashtag vignette.
Note that this TAGS tracker is read-only in the web browser, because the utility of tidytags is reading a TAGS tracker archive into R using
read_tags("18clYlQeJOc6W5QRuSlJ6_v3snqKJImFhU42bRkM_OX8") and then conducting analyses in an R environment.
- Before you do a pull request, you should always file an issue and make sure someone from the tidytags team agrees that it’s a problem, and is happy with your basic proposal for fixing it. We don’t want you to spend a bunch of time on something that we don’t think is a real problem or an appropriate solution.
- Also make sure to read the tidyverse style guide which will make sure that your new code and documentation matches the existing style. This makes the review process much smoother.
You are welcome to contribute a pull request (PR) to tidytags. The most important thing to know is that tidyverse packages use roxygen2: this means that documentation is found in the R code close to the source of each function.
Fork, clone, branch
The first thing you’ll need to do is to fork the tidytags GitHub repo, and then clone it locally. We recommend that you create a branch for each PR.
Before changing anything, make sure the package still passes the below listed flavors of
R CMD check locally for you.
goodpractice::goodpractice(quiet = FALSE, ) devtools::check()
Match the existing code style. This means you should follow the tidyverse style guide. Use the styler package to apply the style guide automatically and the spelling package to check spelling.
Be careful to only make style changes to the code you are contributing. If you find that there is a lot of code that doesn’t meet the style guide, it would be better to file an issue or a separate PR to fix that first.
styler::style_pkg() spelling::spell_check_package() spelling::update_wordlist()
We use roxygen2, specifically with Markdown syntax, to create
NAMESPACE and all
.Rd files. All edits to documentation should be done in roxygen comments above the associated function or object. Then, run
devtools::document() to rebuild the
RoxygenNote in DESCRIPTION for the version of roxygen2 being used.
We use testthat for testing. Contributions with test cases are easier to review and verify.
Note that because tidytags queries the Twitter API, testing can be a bit tricky. Be sure to follow the Getting started with tidytags vignette for establishing your own Twitter API tokens to conduct local testing. For CI testing, view the setup-tidytags.R file in the package testing documentation to see how fake OAuth tokens are set up. The HTTP testing in R book is an invaluable resource.
Before submitting your changes, make sure that the package either still passes
R CMD check, or that the warnings and/or notes have not changed as a result of your edits.
devtools::check() goodpractice::goodpractice(quiet = FALSE)
When you’ve made your changes, write a clear commit message describing what you’ve done. If you’ve fixed or closed an issue, make sure to include keywords (e.g.
fixes #17) at the end of your commit message (not in its title) to automatically close the issue when the PR is merged.
Push and pull
Once you’ve pushed your commit(s) to a branch in your fork, you’re ready to make the pull request. Pull requests should have descriptive titles to remind reviewers/maintainers what the PR is about. You can easily view what exact changes you are proposing using either the Git diff view in RStudio, or the branch comparison view you’ll be taken to when you go to create a new PR. If the PR is related to an issue, provide the issue number and slug in the description using auto-linking syntax (e.g.
Check the docs
Double check the output of the GitHub Actions CI for any breakages or error messages.
- Happy Git and GitHub for the useR by Jenny Bryan.
- Contribute to the tidyverse covers several ways to contribute that don’t involve writing code.
- Contributing Code to the Tidyverse by Jim Hester.
- R packages by Hadley Wickham.
NEWS.mdis a good source of examples for both content and styling.
- Closing issues using keywords on GitHub.
- Autolinked references and URLs on GitHub.
- GitHub Guides: Forking Projects.