Skip to content

Latest commit

 

History

History
198 lines (121 loc) · 13.6 KB

CONTRIBUTING.md

File metadata and controls

198 lines (121 loc) · 13.6 KB

Introduction

You're the Hero we all Need but don't Deserve

First off, thank you for considering contributing to Health Sense. It's people like you that make Health Sense such a great tool.

[source: Health Sense] Need more inspiration? [1]read the docs [2]mustache.js

Guidelines

Following these guidelines helps to communicate that you respect the time of the developers managing and developing this open source project. In return, they should reciprocate that respect in addressing your issue, assessing changes, and helping you finalize your pull requests.

[source: Hoodie]

Contributions Types

Keep an open mind! Improving documentation, bug triaging, or writing tutorials are all examples of helpful contributions that mean less work for you.

Elasticsearch 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 which can be incorporated into Elasticsearch itself.

[source: Elasticsearch] Need more inspiration? [1]devise [2]geocoder (“known issues”)

Contributions we are NOT looking for

Again, defining this up front means less work for you. If someone ignores your guide and submits something you don’t want, you can simply close it and point to your policy.

Please, don't use the issue tracker for [support questions]. Check whether the #pocoo IRC channel on Freenode can help with your issue. If your problem is not strictly Werkzeug or Flask specific, #python is generally more active. Stack Overflow is also worth considering.

[source: Flask] Need more inspiration? [1]cucumber-ruby [2]read the docs

Ground Rules

Better Transparency

This includes not just how to communicate with others (being respectful, considerate, etc) but also technical responsibilities (importance of testing, project dependencies, etc). Mention and link to your code of conduct, if you have one.

Responsibilities

  • Ensure cross-platform compatibility for every change that's accepted. Windows, Mac, Debian & Ubuntu Linux.
  • Ensure that code that goes into core meets all requirements in this checklist:Link
  • Create issues for any major changes and enhancements that you wish to make. Discuss things transparently and get community feedback.
  • Don't add any classes to the codebase unless absolutely needed. Err on the side of using functions.
  • Keep feature versions as small as possible, preferably one new feature per version.
  • Be welcoming to newcomers and encourage diverse new contributors from all backgrounds. See the Python Community Code of Conduct.

[source: cookiecutter] Need more inspiration? [1]celery [2]geocoder

Your First Contribution

Help people who are new to your project understand where they can be most helpful. This is also a good time to let people know if you follow a label convention for flagging beginner issues.

Unsure where to begin contributing to the project? You can start by looking through these beginner and help-wanted issues: Beginner issues - issues which should only require a few lines of code, and a test or two. Help wanted issues - issues which should be a bit more involved than beginner issues. Both issue lists are sorted by total number of comments. While not perfect, number of comments is a reasonable proxy for impact a given change will have.

Bonus points: You can go through the Pinned Issues section

Here are a couple of friendly tutorials you can include: http://makeapullrequest.com/ and http://www.firsttimersonly.com/

Working on your first Pull Request? You can learn how from this free series, How to Contribute to an Open Source Project on GitHub.

[source: React]

As a side note, it helps to use newcomer-friendly language throughout the rest of your document. Here are a couple of examples from Health Sense:

At this point, you're ready to make your changes! Feel free to ask for help; everyone is a beginner at first 😸

If a maintainer asks you to "rebase" your PR, they're saying that a lot of code has changed, and that you need to update your branch so it's easier to merge.

Getting started

How to submit a contribution

How you write this is up to you, but some things you may want to include:

  • We use Github to manage issues and collaborate accordingly.
  • We use various other tools for managing the project.

For something that is bigger than a one or two line fix:

1.  Create your own fork of the code
2.  Do the changes in your fork
3.  If you like the change and think the project could use it:

 * Be sure you have followed the code style for the project.

Small or "obvious" fixes

Small contributions such as fixing spelling errors, where the content is small enough to not be considered intellectual property, can be submitted by a contributor as a patch.

As a rule of thumb, changes are obvious fixes if they do not introduce any new functionality or creative thinking. As long as the change does not affect functionality, some likely examples include the following:

  • Spelling / grammar fixes
  • Typo correction, white space and formatting changes
  • Comment clean up
  • Bug fixes that change default return values or error codes stored in constants
  • Adding logging messages or debugging output
  • Changes to ‘metadata’ files like .gitignore, .swiftLint, .github, build scripts, etc.
  • Moving source files from one directory or package to another

[source: Chef] Need more inspiration? [1]puppet

How to report a bug

Explain security disclosures first

At bare minimum, include this sentence:

If you find a security vulnerability, do NOT open an issue. Email our project manager instead.

If you don’t want to use your personal contact information, set up a “security@” email address. Larger projects might have more formal processes for disclosing security, including encrypted communication. (Disclosure: I am not a security expert.)

Any security issues should be submitted directly to [email protected] In order to determine whether you are dealing with a security issue, ask yourself these two questions:

  • Can I access something that's not mine, or something I shouldn't have access to?
  • Can I disable something for other people?

If the answer to either of those two questions are "yes", then you're probably dealing with a security issue. Note that even if you answer "no" to both questions, you may still be dealing with a security issue, so if you're unsure, just email us at [email protected].

[source: Travis CI] Need more inspiration? [1]celery [2]express.js

File a bug report

You can even include a template so people can just copy-paste (again, less work for you).

When filing an issue, make sure to follow the Issue template and answer as much information as possible for easy communication between collaborators.

[source: Go] Need more inspiration? [1]celery [2]atom (includes template)

How to suggest a feature or enhancement

If you have a particular roadmap, goals, or philosophy for development, share it here.

This information will give contributors context before they make suggestions that may not align with the project’s needs.

The Express philosophy is to provide small, robust tooling for HTTP servers, making it a great solution for single page applications, web sites, hybrids, or public HTTP APIs.

Express does not force you to use any specific ORM or template engine. With support for over 14 template engines via Consolidate.js, you can quickly craft your perfect framework.

[source: Express] Need more inspiration? Health Sense

Explain your desired process for suggesting a feature

If there is back-and-forth or sign-off required, say so. Ask them to scope the feature, thinking through why it’s needed and how it might work.

If you find yourself wishing for a feature that doesn't exist in Elasticsearch, you are probably not alone. There are bound to be others out there with similar needs. Many of the features that Elasticsearch has today have been added because our users saw the need. Open an issue on our issues list on GitHub which describes the feature you would like to see, why you need it, and how it should work.

[source: Elasticsearch] Need more inspiration? [1]hoodie [2]ember.js

Code review process

Explain how a contribution gets accepted after it’s been submitted

Who reviews it? Who needs to sign off before it’s accepted? When should a contributor expect to hear from you? How can contributors get commit access, if at all?

The core team looks at Pull Requests on a regular basis in a weekly triage meeting that we hold in a public Google Hangout. The hangout is announced in the weekly status updates that are sent to the puppet-dev list. Notes are posted to the Puppet Community community-triage repo and include a link to a YouTube recording of the hangout. After feedback has been given we expect responses within two weeks. After two weeks we may close the pull request if it isn't showing any activity.

[source: Puppet] Need more inspiration? [1]meteor [2]express.js

Community

If there are other channels you use besides GitHub to discuss contributions, mention them here. You can also list the author, maintainers, and/or contributors here, or set expectations for response time.

You can chat with the core team on https://gitter.im/cucumber/cucumber. We try to have office hours on Fridays.

[source: cucumber-ruby] Need more inspiration? [1]chef [2]cookiecutter

BONUS: Code, commit message and labeling conventions

These sections are not necessary, but can help streamline the contributions you receive.

Explain your preferred style for code, if you have any

Need inspiration? [1]requirejs [2]elasticsearch

Explain if you use any commit message conventions

Please try to label your commit messages and description as much as possible.

  • feat: A new feature
  • fix: A bug fix
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, for matting, missing semi-colons, etc)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • chore: Unrelated to code or removing (white-space, commented code, formatting, missing semi-colons, etc)
  • build: Build related changes

Need inspiration? [1]angular [2]node.js

Explain if you use any labeling conventions for issues

Need inspiration? [1]standardissuelabels [2]atom