-
-
Notifications
You must be signed in to change notification settings - Fork 102
Contributing Guide
Thanks for your interest in contributing to TTR!
Please do not open an issue to ask a question about how to use the package. Instead, ask your question on Stack Overflow or the R-SIG-Finance mailing list (you must subscribe to post).
- Determine which repository the bug/feature should be reported in. The process of creating a minimal, reproducible example should identify the package that contains the bug or should contain the feature. Please email the maintainer if you're unsure where to create an issue.
- Search current open GitHub issues to check if the bug/feature has already been reported/requested.
- Ensure your fork and local copy are up-to-date, and verify the bug still exists in the HEAD of the master branch.
- If the bug exists in the HEAD of master, and you can't find an open issue,
then open a new issue.
Please be sure to:
- Use an informative and descriptive title,
- Describe the expected behavior and why you think the current behavior is a bug.
- Include as much relevant information as possible; at minimum:
- a minimal, reproducible example
- the output from
sessionInfo()
- Changes that are purely cosmetic in nature (e.g. whitespace changes, code formatting, etc) will generally not be accepted because they do not add to the stability, functionality, or testability of the project.
- Unless the change is extremely trivial (e.g. typos), please create an issue and wait for feedback before you start work on a pull request. That will avoid the possibility you spend time on a patch that won't be merged.
- Create a branch for the feature/bug fix reported in the issue. Please use a
short and descriptive branch name that starts with the issue number (e.g.
123_custom_function). Use that branch as the base for your pull request.
Pull requests on your version of
master
will not be accepted, because they can make it difficult for you to update your fork if your pull request isn't incorporated verbatim. - A pull request should only be for one issue, so please
git rebase -i
and squash the commits on your feature branch into one commit before creating the pull request. Please usegit commit --amend
to amend your commit if you are asked to make changes. It's okay to force update your pull request withgit push --force
. - Please write a great commit message.
- It would be much appreciated if you also add tests that cover your changes.
Follow the The Seven Rules of How to Write a Git Commit Message. Pay particular attention to rule 7: Use the body to explain what and why versus how. The body should also include the motivation for the change and how it compares to prior behavior.
If the commit is to fix a bug or add a feature, the commit message should contain enough information to understand the bug/feature without having to reference an external tracker (e.g. GitHub issues). But please do reference the GitHub issue on the last line of your commit message body. For example, here is a great xts commit message:
Correct endpoints when index is before the epoch
The endpoints C code casts the double index to long, which truncates
it toward zero. This behavior is desired when the index is positive,
because it moves the endpoint *back* in time. But when the index is
negative, truncating toward zero moves the endpoint *forward* in time.
This is also an issue if the index value is stored as integer, since the
C99 specification states that integer division truncates toward zero.
If the first index value is less than zero, branch into a special case
to handle pre-epoch index values. This avoids performance degradation
if all index values are after the epoch.
If the index value is less than zero, simply add 1 to offset the
truncation toward zero. We also need to further adjust the potential
endpoint value if the index is exactly equal to zero.
Fixes #144.
- The data.table Contributing Guide.
- The Atom Contributing Guide.
- How to create a minimal, reproducible example.
- How to Write a Git Commit Message.
- The Mercurial Contributing Guide.
- The Hugo Contributing Guide.