Skip to content

Latest commit

 

History

History
112 lines (80 loc) · 4.65 KB

CONTRIBUTING.md

File metadata and controls

112 lines (80 loc) · 4.65 KB
Raw

Contributing

We value your contributions, and we want to make it as easy as possible to work in this repository. One of the first things to decide is which branch to base your work on. If you get confused, just ask and we will help. If a reviewer realizes you have based your work on the wrong branch, we'll let you know so that you can rebase it.

Note: To contribute code to Ritchie projects, see the Ritchie community guidelines as well as the Open source contribution guidelines and our Code of Conduct.

New features for the project

Ritchie is composed of 3 projects which release at different times.

Always base your work on the project's master branch, naming your new branch according to the following guide :

Rit branchs

Examples : feature/name or fix/name

Collaborate on a pull request

Unless the PR author specifically disables it, you can push commits into another contributor's PR. You can do it from the command line by adding and fetching their remote, checking out their branch, and adding commits to it. Even easier, you can add commits from the Github web UI, by clicking the pencil icon for a given file in the Files view.

If a PR consists of multiple small addendum commits on top of a more significant one, the commit will usually be "squash-merged", so that only one commit is merged in. On occasion this is not appropriate and all commits will be kept separate when merging.ficant one, the commit will usually be "squash-merged", so that only one commit is merged in. On occasion this is not appropriate and all commits will be kept separate when merging.

Pull request guidelines

Help us review your PRs more quickly by following these guidelines.

  • Try not to touch a large number of files in a single PR if possible.

  • Provide a clear description of the motivation of your PR, this is a large project, so context is important

  • Don't change whitespace or line wrapping in parts of a file you are not editing for other reasons. Make sure your text editor is not configured to automatically reformat the whole file when saving.

  • Reviewers will check the staging site and contact you to fix eventual problems.

  • If you agree with the suggested comment, just resolve or emoji it. No need to write a confirmation. The code owner's mailbox will thank you later :D

If you can think of other ways we could streamline the review process, let us know.

Pull review guidelines

For those wanting to contribute on the quality of the incoming code, try to follow these suggestions for a happy community =]

  • Be suggestive, never impose a correction or criticize your peer. Instead of "change this code",go for more like "what do you think about implementing this way?"

  • Explain why you suggested such correction, a change without meaning might not be productive. The author has all the right to counterargue a comment if she thinks it is for the best of the project. Provide a clear justification technical or business and even links or references if possible. Everybody loves to learn something new about coding to become a better developer.

  • Sometimes the literal answer might not be necessary. Instead of pasting the solution in verbatim, provide the right direction and let the author figure out.

  • Always have a sense of community and try to help others, because they are trying to help us.

  • If a debate gets more heated in a review, try to set a call or meeting to clarify points, because in a meeting it is possible to talk and understand each other better than with texts in a pr review.

  • Requesting changes sometimes is perceived as a harsh action for some engineers. Try to do it with parsimony, usually when you spot a production-breaking change or to prevent an already approved PR to be merged without that last important touch on code.

Style guide

Ritchie does not currently maintain a style guide. Use your best judgment, and try to follow the example set by the existing files.

Hacktoberfest

  • If you want to contribute with something thay doesn't have any ISSUE yet, you can create a new detailed ISSUE for our team to evaluate, and they will eventually add a Hackoberfest label to allow you to participate the event resolving this ISSUE.

  • Formulas created for the Hacktoberfest will be submitted to our team review before validation for the Zup Hacktoberfest event.