Skip to content

Latest commit

 

History

History
149 lines (91 loc) · 11.8 KB

CONTRIBUTING.md

File metadata and controls

149 lines (91 loc) · 11.8 KB

Contributors Guide

Welcome to the Contributors' Guide! Thank you so much for considering contributing to this repository. We really appreciate all the contributors who try to make this space better. This document serves as a guide that will show you the path of least resistance for making valuable contributions to this project. Reading and following this guide shows that you respect the time of the developers managing and developing this space. In return, they will reciprocate that respect by addressing your issue, assessing changes, and helping you finalize your pull requests. This is the only way we can produce anything of value.

We take our Code of Conduct very seriously, and all contributors must abide by it. Let’s treat each other with respect and remember we are all people who care and are trying.

Getting Started

Before you begin:

If you're looking to start learning how to work with this repository, this document tries to provide all the resources to get you from confusion to productivity.

We're not here to reinvent the wheel, so where we feel appropriate, we reference and link to resources we feel do a good job of helping a newbie through that particular section of the setup.

Most of the documents are currently written using Markdown to make it easy to add, modify, and edit what we need to.

We try not to take anything for granted, but in case we miss something, please reach out to the maintainers and let us know!

We really want to make space for all developers to feel comfortable and supported while contributing, and documents like this provide a valuable resource for them to use.

Table of Contents

What type of contributions we're looking for

This is an open source project, and we love to receive contributions from our community — you! There are many ways to contribute, from writing tutorials or blog posts, improving the documentation, submitting bug reports and feature requests, or writing code that can be incorporated into the application codebase itself.

Asking to Pair

This is a beginner friendly repository. Contributors can request a pairing (zoom/google hangouts) to pair with a maintainer on tackling an issue, submitting a PR, etc. If you want to do it for a pre-existing issue, simply indicate so in a comment under that issue. If not, you may open a new Question issue and request a pairing. Please understand that this is a volunteer effort; maintainers may not always be available for pairing.

The Basics

Setting up a Github Account

We're going to assume you know what Github is (otherwise, how would you be reading this?).

If you haven't made a GitHub account yet, make one now. It's free!

Discussions

For questions, comments, or other discussion topics that don’t have a clear action, please use the discussion board.

For example, if you want to ask “Should we do…” or “How do I do…”, you’ll start in the discussion board. This allows a whole-group conversation, and we can integrate the answer into documents or issues as necessary.

Working With Issues

GitHub uses a feature called issues. Issues are essentially a way to highlight bugs, features, problems, or any sort of suggestion or action you wish to happen on a GitHub hosted project (you can find a more comprehensive explanation here).

We highly recommend looking at the existing repository issues to find a good open issue to start with. We always try to keep them populated with some beginner-friendly issues that anyone can attempt to solve.

If you find an issue you would like to work on, comment on the issue and a maintainer will respond and/or assign it to you. It can't be assigned to you unless you've commented on the issue.

We also use a PR Submitted tag to indicate when a Pull Request has been submitted for an issue, but it hasn't yet been merged, as most people would rather do on an issue with no attempted PRs yet.

If you feel like there's a contribution you would like to make that isn't represented by an already existing issue, feel free to create your own! We have four issue templates that you can use, which provide some (hopefully) helpful guidelines on how to form your issues. They are:

  • FEATURE
  • BUG
  • QUESTION
  • DOCUMENTATION

If you feel like you have an issue that doesn't match those categories though, go ahead and ignore the templates and create an issue the old fashioned way.

Local development

Follow the steps for local development in our README

Creating a Pull Request

⚠️ Heads up! If you'd like to make a Pull Request, please make sure you've created an issue (or a discussion board post) first, and that you've been assigned to the issue. This allows the maintainer team to provide guidance and prioritize tasks — otherwise you may run the risk of spending time on something that doesn't end up getting accepted for various reasons. We appreciate everyone’s efforts, and want to ensure that our communication is clear and open as much as possible. We strongly encourage communication and clarity before doing work.

Once you've been assigned the issue, you have two options.

The first option is using the GitHub interface to fork the repository, making an edit right here in the github client, and then submit a Pull Request (no code or terminals or IDE required). This guide shows just how to do that for a small personal repo. You would simply replace the step of creating a new repository to just navigating to this one and forking that instead. This is a great idea if you simply plan to add to or edit one of the markdown files we use for documentation in this project.

The second option is to go the traditional route of forking the repository, creating a local copy of that fork, and working on your changes that way. This is also the only way to go if this project expands to include an associated application. For that, we recommend this guide.

Note: The guide referenced above uses master as the naming convention for the default branch in all its repositories. In this project, main is the default branch name. When following the instructions in the guide, simply replace master with main wherever it appears, and it should proceed as normal. This guide from Scott Hanselman can help you transition your own personal repositories from master to main, and explains the methodology as to why you should.

Awaiting Review

Once you've submitted your pull request, the only thing left is to wait for feedback from one of the project maintainers. Since this is volunteer work for all of course, we ask for patience if you don't see a response immediately. Sometimes it takes about a day for someone's schedule to clear up to have the sit to properly review incoming PRs. We'd rather not rush a response after someone has put time and effort into submitting it. If it's been over one week and you haven't received any acknowledgement, you can post a comment on your PR reminding of it's status.

The purpose of reviews is to create the best experience we can for our contributors.

  • Reviews are always respectful, acknowledging that everyone did the best possible job with the knowledge they had at the time.
  • Reviews discuss content, not the person who created it.
  • Reviews are constructive and start a conversation around feedback.

If the PR looks good, a maintainer will typically give feedback and merge the request immediately, otherwise they'll let you know what questions they have or what needs to change before your work can be accepted. Once it is, you'll see your changes on the main branch and VOILA, your open source contribution is complete!

Suggested Changes

We may ask for changes to be made before a PR can be merged, either using suggested changes or pull request comments. You can apply the suggested changes directly through the UI. You can make any other changes in your fork, then commit them to your branch.

As you update your PR and apply changes, mark each conversation as resolved.

Tips and Gotchas

This section is just little notes and bits of information that can smooth over some of the bumps and hiccups that can come along with contributing.

  • While this isn't absolutely required, we highly recommend associating your Pull Requests with the Issue that they're intended to address. This makes review much easier and avoids confusion when looking back at past commits. GitHub allows you to link a PR to an issue both during and after the PR's creation (the option should be located in the rightmost panel of the GitHub PR interface).
  • Please do not hesitate to ask for help in any part of this process if you feel confused. As soon as they can, project maintainers can try and get you through the parts which are confusing you. Just be aware that no one here is a GitHub expert, we're just folks happy and willing to help others get some experience.
  • Be extra careful when working with git in the command line. Incorrectly typed names or commands can have strange results, and navigating git issues can be very perplexing. Fortunately, there's a plethora of resources on fixing said issues, and rest assured that any error you make has already been done, and solved, by someone.

That's all for now. If you feel like anything is missing from this document that you wish were included, let us know. Or hey, open up a fresh issue and take a shot at helping us make it better!

Reporting a bug

We have a bug report template already created for your use. Just select it when opening up an issue on github and follow the headers to formulate your issue.

When filing an issue, make sure to answer these three questions:

  • What did you do?
  • What did you expect to see?
  • What did you see instead?

Labelling Conventions

This repository has three basic types of labels:

  • Type - The nature of the issue. BUG, FEATURE, QUESTION, or DOCUMENTATION (will be pre-assigned if using issue templates). Only one active per issue
  • Status - What part of the process is this issue in. e.g. Active, Needs Review, Resolved, etc. (will be filled in by maintainers and some contributors). Only one active per issue.
  • Context - Additional information that helps people parse issues. e.g. "good first issue", "for maintainer only". Multiple may be active on one issue. Footer © 2022 GitHub, Inc. Footer navigation