-
Notifications
You must be signed in to change notification settings - Fork 9
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Added contribution directions as well as a code of conduct. The READM…
…E now is capable of listing the various levels of contribution in addition to contributors.
1 parent
918127c
commit 1cd3805
Showing
6 changed files
with
173 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
# Contributor Code of Conduct | ||
|
||
As contributors and maintainers of this project, we pledge to respect all people who contribute through reporting issues, posting feature requests, updating documentation, submitting pull requests or patches, and other activities. | ||
|
||
We are committed to making participation in this project a harassment-free experience for everyone, regardless of level of experience, gender, gender identity and expression, sexual orientation, disability, personal appearance, body size, race, ethnicity, age, or religion. | ||
|
||
Examples of unacceptable behavior by participants include the use of sexual language or imagery, derogatory comments or personal attacks, trolling, public or private harassment, insults, or other unprofessional conduct. | ||
|
||
Project maintainers have the right and responsibility to remove, edit, or reject comments, commits, code, wiki edits, issues, and other contributions that are not aligned to this Code of Conduct. Project maintainers who do not follow the Code of Conduct may be removed from the project team. | ||
|
||
Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by opening an issue, or contacting emailing the project lead (<ross.a.drucker@gmail.com>). | ||
|
||
This Code of Conduct is adapted from the [Contributor Covenant](http:contributor-covenant.org), version 1.0.0, available at https://www.contributor-covenant.org/version/1/0/0/code-of-conduct.html |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,94 @@ | ||
# Contributing to `sportyR` | ||
|
||
First of all, thanks for considering contributing to `sportyR`! The package, just like all open source projects, can't improve without contributions and feedback from its users. | ||
|
||
`sportyR` is a labor of love, and no one is being paid to work on it. Please keep this in mind throughout your contributions, as responses and bug fixes may take a bit of time to work through. | ||
|
||
## Play by the Rules - Code of Conduct | ||
|
||
Please note that this project is released with a [Contributor Code of Conduct](CODE-OF-CONDUCT.md). By participating in this project, you're agreeing to abide by its terms. | ||
|
||
`sportyR`'s League Office (see below) reserves the right to limit and/or exclude contributions from those who violate the Code of Conduct as is appropriate. | ||
|
||
## Getting In The Game - Making Contributions | ||
|
||
There's many ways that you can contribute to `sportyR`. If you want a more complete explanation of how you can contribute (to any open source project, not just this one), check out [this guide](https://opensource.guide/how-to-contribute/). | ||
|
||
### Be A Fan - Love the Package | ||
|
||
Think that `sportyR` is awesome? Want to share the awesome plots and gifs you made with it? Let others discover it by telling them in person, tagging the project on [Twitter](https://www.twitter.com/sportyR_pkg), or writing a blog post about your experience with it! | ||
|
||
If you're using `sportyR` in an academic context, please do your best to cite the package as well. | ||
|
||
### Scouting Reports - Bugs and Feature Suggestions | ||
|
||
Like any good sports organization, `sportyR` does its best to scout for itself and solve problems before they become problems. Given a tough test, problems are sure to arise. They can be overcome in their next match, however, by making use of good scouting (bug) reports. Please create an [Issue](https://github.com/rossdrucker/sportyR/issues) when you find something that needs to be fixed. Here's a good general guide to follow for creating one: | ||
|
||
1) Please double check the [README](https://www.github.com/rossdrucker/sportyR) and the [Issues](https://github.com/rossdrucker/sportyR/issues) pages to be sure that your problem hasn't already been solved. This isn't meant to stop you from reporting your problem, but rather to be sure that it's a new one so it can be properly addressed. | ||
|
||
2) Title your issue with a clear description of what you're looking for. Titling your Issue "This is weird..." doesn't give as good of an idea of what you're after as titling your Issue "NCAA Football hashmarks are out of place." Even better if you can prefix your Issue with `bug` or `feature-request` | ||
|
||
3) Spend some time creating a detailed explanation, with a reproducible example, of the problem you're having. Without being able to understand the issue, it's even harder to find what's causing it to occur. Do your best to make sure your code is well-formatted and easy for others to read: | ||
|
||
- Make sure you name your variables something informative | ||
|
||
- Use comments to indicate where you believe the problem in your code is occurring | ||
|
||
- Only include the necessary code to recreate the problem in your Issue report so it can be solved correctly | ||
|
||
### The League Office Directory - Contributor Status | ||
|
||
Want to join `sportyR`'s team and help shape the package? Great! We're always recruiting! Here's how you can join: | ||
|
||
- **League Office** — By contributing regularly, providing substantial upgrades to the package's overall structure and function, and helping to maintain the package across the board, you'll be considered for membership in the `sportyR`'s **League Office**. | ||
|
||
In order to join the League Office, you must be invited by a current member and get approved by a majority of the other members of the League Office | ||
|
||
When you become a League Officer, you should list yourself in the [README](https://github.com/rossdrucker/sportyR), and also list yourself in the [DESCRIPTION](https://github.com/rossdrucker/sportyR/blob/master/DESCRIPTION) file with the role of `"aut"` | ||
|
||
- **General Managers** — Writing the code that extends `sportyR`'s sport-plotting reach by developing for new sports will earn you a spot as a **General Manager**. Please list yourself (in alphabetical-by-sport order) in the [README](https://github.com/rossdrucker/sportyR) with what sport you've added. | ||
|
||
Note that by contributing a new sport, you'll be considered the General Manager of that sport. You may be asked to assist in sport-specific issues if the League Office is unable to provide a clear answer | ||
|
||
- **Coaching Staffs** — Patching a bug in an already-existing sport means you're eligible to be on that sport's **coaching staff**. You've got your eye on the game, and it's much appreciated! Coaches can list themselves in the README as well, similar to how General Managers are listed | ||
|
||
- **Scout Team** — By fixing a typo, making a slight modification to something that already exists in `sportyR`, or regularly reporting issues with the package that you encounter (or even helping others solve theirs), you can join the **Scout Team**. Thanks for being on the front lines of the package's operation! Feel free to list yourself in the [README](https://github.com/rossdrucker/sportyR) under **Scout Team** with a brief description of your role ("Typo Fixer" or "Question Answerer" work great, but be as creative with it as you'd like) | ||
|
||
## Becoming a Coach/General Manager - Pull Request Process | ||
|
||
To become a Cocah or General Manager, submit a [Pull Request](https://github.com/rossdrucker/sportyR/pulls) (PR) on GitHub using this process: | ||
|
||
1) Create a branch in git and make the changes you wish to see implemented | ||
|
||
2) Push your branch to GitHub and issue your PR. Your PR should be titled in a way that it's obvious what it's overarching goal is | ||
|
||
3) Discuss the PR in as much detail as possible. Provide links to your sources, and clearly explain why the PR is necessary and how it can be used. Any deviations from the conventions of the rest of the package should be clearly explained, but only if the deviation is necessary | ||
|
||
4) Continue iterating your request until the League Office determines if the addition is a good fit for `sportyR` | ||
|
||
Your request will be checked against the following points: | ||
|
||
- **Motivation** — Clearly and concisely explain what motivated the PR. If it addresses a particular issue, be sure you link to it | ||
|
||
- **Only Related Changes** — Before you submit your PR, make sure your PR only addresses *one* issue. If you want to fix multiple issues, please address them in separate PRs so that it's clear what change is being made in each PR. Please also be sure that your PR doesn't introduce any new bugs or break any existing code | ||
|
||
- **Code Style and File Naming** — Please follow [the tidyverse styleguide](https://style.tidyverse.org/), with the only deviations being that assignment should be done through the `=` operator instead of the `<-` operator, and explicit `return()` calls to return what's needed from each function. Maintaining consistent styling facilitates easier current maintenance and future development. | ||
|
||
Additionally, please keep consistent with the naming of new files, especially when adding a new sport. Files that contain a new league should be named as `{sport}-{league}-features.R`, and if this is a new sport entirely, be sure to add the corresponding `sport-{new sport here}.R` file as well. Although leagues in the same sport may share different features (and therefore could potentially share feature functions), this more granular and stratified approach allows for easier maintenance and control should any given league change up their surface's dimensions | ||
|
||
- **Documentation** — When introducing new code, please make sure you document it appropriately. Documentation comes in two main forms: `roxygen2` documentation for the functions you write, and comments throughout the code that explain what your code is trying to achieve. Both types of documentation are very important! Be sure to run `devtools::document()` to be sure your documentation files are properly created before submitting, and be sure to check for spelling and grammar. | ||
|
||
A quick note on internal/helper function documentation: please make sure that you omit these files from the package build by listing them in [`.Rbuildignore`](.Rbuildignore). This reduces the overall size of the package when being submitted to CRAN, which only allows a maximum package size of 5MB. Reach out if you need help doing this | ||
|
||
- **Testing** — `sportyR` relies upon extensive testing to ensure that the package runs smoothly. It currently has 100% code coverage, and your PR should ensure that this remains the case. | ||
|
||
To test your coverage, be sure to have the `covr` package installed and run the following commands: | ||
|
||
``` | ||
cov = covr::package_coverage() | ||
covr::zero_coverage(cov) | ||
``` | ||
This will show you where your new code lacks sufficient coverage, so please be sure to add tests in the `tests/` repository to make sure you're covered! For help on creating tests (especially of the visual variety), please reach out to the League Office | ||
Although this seems like a lot of work and a big hassle, the League Office doesn't expect perfection and is on hand to help out as needed. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters