From 1be821b07a4764fc00dc9a967d41f671d08d6708 Mon Sep 17 00:00:00 2001 From: David Sparer Date: Thu, 20 Oct 2016 11:30:49 -0600 Subject: [PATCH 1/6] added rough draft of the contribution intro --- .github/CONTRIBUTING.md | 193 +++++++++++++++++++++++++++++++++++++++- 1 file changed, 192 insertions(+), 1 deletion(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index ee09427e..b0656356 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -1 +1,192 @@ -Please see the [Wiki](https://github.com/mRemoteNG/mRemoteNG/wiki/Development) for contributing information. +# Welcome! + +We are really glad you are interested in contributing to mRemoteNG! + +Open source software is best when shared with others. This also applies to the work that goes into the software. Your ideas and passion are what make this software great. + + +### Ways you can contribute +There are many ways that you can help improve mRemoteNG, even if you don't know how to program. + +For example, you might: +- add documentation or "how-to" articles on the Wiki +- answer support questions on the [forum](http://forum.mremoteng.org) +- add or improve a translation +- submit a pull request for a bug or feature ticket + + +***========= Everything below this line is template =============*** + + +### Tell them why they should read your 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](https://github.com/hoodiehq/hoodie/blob/master/CONTRIBUTING.md)] + +### Explain what kinds of contributions you are looking for. + +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](https://github.com/elastic/elasticsearch/blob/master/CONTRIBUTING.md)] **Need more inspiration?** [1] [Devise](https://github.com/plataformatec/devise/wiki/Contributing) [2] [Geocoder](https://github.com/alexreisner/geocoder#known-issue) (“known issue”) + +### Explain contributions you are NOT looking for (if any). + +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](https://github.com/pallets/flask/blob/master/CONTRIBUTING.rst)] **Need more inspiration?** [1] [cucumber-ruby](https://github.com/cucumber/cucumber-ruby/blob/master/CONTRIBUTING.md#about-to-create-a-new-github-issue) [2] [Read the Docs](http://read-the-docs.readthedocs.org/en/latest/open-source-philosophy.html#unsupported) + +# Ground Rules +### Set expectations for behavior (yours, and theirs). +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: https://gist.github.com/audreyr/4feef90445b9680475f2 +* 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](https://www.python.org/psf/codeofconduct/). + +[source: [cookiecutter](https://github.com/audreyr/cookiecutter/blob/master/CONTRIBUTING.rst)] **Need more inspiration?** [1] [Celery](https://github.com/celery/celery/blob/master/CONTRIBUTING.rst#community-code-of-conduct) [2] [geocoder](https://github.com/alexreisner/geocoder#contributing) + +# 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 Atom? 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. + +[source: [Atom](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#your-first-code-contribution)] **Need more inspiration?** [1] [Read the Docs](http://docs.readthedocs.org/en/latest/contribute.html#contributing-to-development) [2] [Django](https://docs.djangoproject.com/en/dev/internals/contributing/new-contributors/#first-steps) (scroll down to "Guidelines" as well) + +### Bonus points: Add a link to a resource for people who have never contributed to open source before. +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](https://egghead.io/series/how-to-contribute-to-an-open-source-project-on-github). + +[source: [React](https://github.com/facebook/react/blob/master/CONTRIBUTING.md#pull-requests)] + +As a side note, it helps to use newcomer-friendly language throughout the rest of your document. Here are a couple of examples from [Active Admin](https://github.com/activeadmin/activeadmin/blob/master/CONTRIBUTING.md): + +>At this point, you're ready to make your changes! Feel free to ask for help; everyone is a beginner at first :smile_cat: + +>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 +### Give them a quick walkthrough of how to submit a contribution. +How you write this is up to you, but some things you may want to include: + +* Let them know if they need to sign a CLA, agree to a DCO, or get any other legal stuff out of the way +* If tests are required for contributions, let them know, and explain how to run the tests +* If you use anything other than GitHub to manage issues (ex. JIRA or Trac), let them know which tools they’ll need to contribute + +>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. + * Sign the Contributor License Agreement, CLA, with the jQuery Foundation. + * Note the jQuery Foundation Code of Conduct. + * Send a pull request indicating that you have a CLA on file. + +[source: [Requirejs](http://requirejs.org/docs/contributing.html)] **Need more inspiration?** [1] [Active Admin](https://github.com/activeadmin/activeadmin/blob/master/CONTRIBUTING.md#1-where-do-i-go-from-here) [2] [Node.js](https://github.com/nodejs/node/blob/master/CONTRIBUTING.md#code-contributions) [3] [Ember.js](https://github.com/emberjs/ember.js/blob/master/CONTRIBUTING.md#pull-requests) + +### If you have a different process for small or "obvious" fixes, let them know. + +> 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, without a CLA. + +>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 Gemfile, .gitignore, build scripts, etc. +* Moving source files from one directory or package to another + +[source: [Chef](https://github.com/chef/chef/blob/master/CONTRIBUTING.md#chef-obvious-fix-policy)] **Need more inspiration?** [1] [Puppet](https://github.com/puppetlabs/puppet/blob/master/CONTRIBUTING.md#making-trivial-changes) + +# 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 XXXX 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 security@travis-ci.org +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 security@travis-ci.org. + +[source: [Travis CI](https://github.com/travis-ci/travis-ci/blob/master/CONTRIBUTING.md)] **Need more inspiration?** [1] [Celery](https://github.com/celery/celery/blob/master/CONTRIBUTING.rst#security) [2] [Express.js](https://github.com/expressjs/express/blob/master/Security.md) + +### Tell your contributors how to 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 answer these five questions: + +>1. What version of Go are you using (go version)? +2. What operating system and processor architecture are you using? +3. What did you do? +4. What did you expect to see? +5. What did you see instead? +> General questions should go to the golang-nuts mailing list instead of the issue tracker. The gophers there will answer or ask you to file an issue if you've tripped over a bug. + +[source: [Go](https://github.com/golang/go/blob/master/CONTRIBUTING.md#filing-issues)] **Need more inspiration?** [1] [Celery](https://github.com/celery/celery/blob/master/CONTRIBUTING.rst#other-bugs ) [2] [Atom](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#reporting-bugs) (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](https://github.com/expressjs/express#philosophy)] **Need more inspiration?** [Active Admin](https://github.com/activeadmin/activeadmin#goals) + +### Explain your desired process for suggesting a feature. +If there is back-and-forth or signoff 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](https://github.com/elastic/elasticsearch/blob/master/CONTRIBUTING.md#feature-requests)] **Need more inspiration?** [1] [Hoodie](https://github.com/hoodiehq/hoodie/blob/master/CONTRIBUTING.md#feature-requests) [2] [Ember.js](https://github.com/emberjs/ember.js/blob/master/CONTRIBUTING.md#requesting-a-feature) + +# 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](https://github.com/puppetlabs/puppet/blob/master/CONTRIBUTING.md#submitting-changes)] **Need more inspiration?** [1] [Meteor](https://meteor.hackpad.com/Responding-to-GitHub-Issues-SKE2u3tkSiH ) [2] [Express.js](https://github.com/expressjs/express/blob/master/Contributing.md#becoming-a-committer) + +# 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](https://github.com/cucumber/cucumber-ruby/blob/master/CONTRIBUTING.md#talking-with-other-devs)] **Need more inspiration?** + [1] [Chef](https://github.com/chef/chef/blob/master/CONTRIBUTING.md#-developer-office-hours) [2] [Cookiecutter](https://github.com/audreyr/cookiecutter#community) + +# 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](http://requirejs.org/docs/contributing.html#codestyle) [2] [Elasticsearch](https://github.com/elastic/elasticsearch/blob/master/CONTRIBUTING.md#contributing-to-the-elasticsearch-codebase) + +### Explain if you use any commit message conventions. + +**Need inspiration?** [1] [Angular](https://github.com/angular/material/blob/master/CONTRIBUTING.md#-git-commit-guidelines) [2] [Node.js](https://github.com/nodejs/node/blob/master/CONTRIBUTING.md#step-3-commit) + +### Explain if you use any labeling conventions for issues. + +**Need inspiration?** [1] [StandardIssueLabels](https://github.com/wagenet/StandardIssueLabels#standardissuelabels) [2] [Atom](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#issue-and-pull-request-labels) From 5131604ba7d52446ce228b8de236cdebfa285ea0 Mon Sep 17 00:00:00 2001 From: David Sparer Date: Thu, 20 Oct 2016 12:33:23 -0600 Subject: [PATCH 2/6] Added drafts for submitting bugs and feature requests --- .github/CONTRIBUTING.md | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index b0656356..451cf3cf 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -15,6 +15,27 @@ For example, you might: - submit a pull request for a bug or feature ticket +# Submitting GitHub Issues +The GitHub Issue tracker is our preferred channel for bug reports and feature requests. +- For questions or general discussion, please use [Gitter Chat](https://gitter.im/mRemoteNG/PublicChat) or the [forum](http://forum.mremoteng.org). + +### Bug reports +A bug is any behavior that does not consistently produce the expected result. + +Guidelines for bug reports: +1. Do not open bug reports for questions. +1. Use the GitHub Issue search to make sure your bug hasn't already been reported. +1. Include as much detailed information as possible. We've included a default template when opening an issue to make this easier. + +### Feature requests +Feature requests are great! Take some time to compose a well thoughtout proposal. It's up to you to convince the project maintainers that your feature is a good idea. To ensure your request receives the consideration that it deserves, include as much detail as possible. For example: +- What is the purpose of the new feature? +- What situation led you to want this feature? +- How does the application perform now and how would the new feature change this? +- If applicable, consider including visual mock-ups to show us what you mean. + + + ***========= Everything below this line is template =============*** From 82848aa4cd31699439468528a05c41508a83b127 Mon Sep 17 00:00:00 2001 From: David Sparer Date: Thu, 20 Oct 2016 15:29:58 -0600 Subject: [PATCH 3/6] Added a section on pull requests --- .github/CONTRIBUTING.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 451cf3cf..2f162cb8 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -34,6 +34,15 @@ Feature requests are great! Take some time to compose a well thoughtout proposal - How does the application perform now and how would the new feature change this? - If applicable, consider including visual mock-ups to show us what you mean. +# Pull requests +Good pull requests are a huge help! If you haven't already, please consider reading [GitHub's guide to contributing to open source](https://guides.github.com/activities/contributing-to-open-source/) + +Our requests when it comes to pull requests: +- Be clear about what your PR seeks to do. +- Keep your PR focused. It should be clear what code changed to achieve your stated goal. +- Add or update tests when possible. Some of the code base is still very hard to effectively test. If you can, please try to create or update tests that relate to your PR. +- Please talk with one of the project maintainers before starting work on large pull requests. Otherwise, you run the risk of putting a lot of time into a feature or refactor that gets denied. + ***========= Everything below this line is template =============*** From 70408ca8eea0a88de088962d6cf0d4a8cf999f13 Mon Sep 17 00:00:00 2001 From: David Sparer Date: Mon, 31 Oct 2016 15:54:42 -0600 Subject: [PATCH 4/6] Finalized the first draft and removed the template lines --- .github/CONTRIBUTING.md | 186 ++-------------------------------------- 1 file changed, 8 insertions(+), 178 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 2f162cb8..6d29bf61 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -17,7 +17,8 @@ For example, you might: # Submitting GitHub Issues The GitHub Issue tracker is our preferred channel for bug reports and feature requests. -- For questions or general discussion, please use [Gitter Chat](https://gitter.im/mRemoteNG/PublicChat) or the [forum](http://forum.mremoteng.org). +- For questions or general discussion, please use [our public Gitter chat](https://gitter.im/mRemoteNG/PublicChat) or the [forum](http://forum.mremoteng.org). +- If you find a security vulnerability, do **NOT** open an issue. Email support@mremoteng.org instead. ### Bug reports A bug is any behavior that does not consistently produce the expected result. @@ -26,13 +27,15 @@ Guidelines for bug reports: 1. Do not open bug reports for questions. 1. Use the GitHub Issue search to make sure your bug hasn't already been reported. 1. Include as much detailed information as possible. We've included a default template when opening an issue to make this easier. +1. Please tag your your bug report with the [`Bug`](https://github.com/mRemoteNG/mRemoteNG/issues?q=is%3Aissue+is%3Aopen+label%3ABug) label. ### Feature requests -Feature requests are great! Take some time to compose a well thoughtout proposal. It's up to you to convince the project maintainers that your feature is a good idea. To ensure your request receives the consideration that it deserves, include as much detail as possible. For example: +Feature requests are great! Take some time to compose a well thought out proposal. It's up to you to convince the project maintainers that your feature is a good idea. To ensure your request receives the consideration that it deserves, include as much detail as possible. For example: - What is the purpose of the new feature? - What situation led you to want this feature? - How does the application perform now and how would the new feature change this? - If applicable, consider including visual mock-ups to show us what you mean. +- Please tag your feature request with the [`Feature Request`](https://github.com/mRemoteNG/mRemoteNG/issues?q=is%3Aissue+is%3Aopen+label%3A%22Feature+Request%22) label. # Pull requests Good pull requests are a huge help! If you haven't already, please consider reading [GitHub's guide to contributing to open source](https://guides.github.com/activities/contributing-to-open-source/) @@ -43,180 +46,7 @@ Our requests when it comes to pull requests: - Add or update tests when possible. Some of the code base is still very hard to effectively test. If you can, please try to create or update tests that relate to your PR. - Please talk with one of the project maintainers before starting work on large pull requests. Otherwise, you run the risk of putting a lot of time into a feature or refactor that gets denied. +# Your first contributions +Want to help but don't know where to start? Check out the issues that we've labeled with [`Help Wanted`](https://github.com/mRemoteNG/mRemoteNG/issues?q=is%3Aissue+is%3Aopen+label%3A%22Help+Wanted%22). These will vary in difficulty, but should be possible for new contributors. - -***========= Everything below this line is template =============*** - - -### Tell them why they should read your 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](https://github.com/hoodiehq/hoodie/blob/master/CONTRIBUTING.md)] - -### Explain what kinds of contributions you are looking for. - -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](https://github.com/elastic/elasticsearch/blob/master/CONTRIBUTING.md)] **Need more inspiration?** [1] [Devise](https://github.com/plataformatec/devise/wiki/Contributing) [2] [Geocoder](https://github.com/alexreisner/geocoder#known-issue) (“known issue”) - -### Explain contributions you are NOT looking for (if any). - -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](https://github.com/pallets/flask/blob/master/CONTRIBUTING.rst)] **Need more inspiration?** [1] [cucumber-ruby](https://github.com/cucumber/cucumber-ruby/blob/master/CONTRIBUTING.md#about-to-create-a-new-github-issue) [2] [Read the Docs](http://read-the-docs.readthedocs.org/en/latest/open-source-philosophy.html#unsupported) - -# Ground Rules -### Set expectations for behavior (yours, and theirs). -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: https://gist.github.com/audreyr/4feef90445b9680475f2 -* 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](https://www.python.org/psf/codeofconduct/). - -[source: [cookiecutter](https://github.com/audreyr/cookiecutter/blob/master/CONTRIBUTING.rst)] **Need more inspiration?** [1] [Celery](https://github.com/celery/celery/blob/master/CONTRIBUTING.rst#community-code-of-conduct) [2] [geocoder](https://github.com/alexreisner/geocoder#contributing) - -# 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 Atom? 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. - -[source: [Atom](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#your-first-code-contribution)] **Need more inspiration?** [1] [Read the Docs](http://docs.readthedocs.org/en/latest/contribute.html#contributing-to-development) [2] [Django](https://docs.djangoproject.com/en/dev/internals/contributing/new-contributors/#first-steps) (scroll down to "Guidelines" as well) - -### Bonus points: Add a link to a resource for people who have never contributed to open source before. -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](https://egghead.io/series/how-to-contribute-to-an-open-source-project-on-github). - -[source: [React](https://github.com/facebook/react/blob/master/CONTRIBUTING.md#pull-requests)] - -As a side note, it helps to use newcomer-friendly language throughout the rest of your document. Here are a couple of examples from [Active Admin](https://github.com/activeadmin/activeadmin/blob/master/CONTRIBUTING.md): - ->At this point, you're ready to make your changes! Feel free to ask for help; everyone is a beginner at first :smile_cat: - ->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 -### Give them a quick walkthrough of how to submit a contribution. -How you write this is up to you, but some things you may want to include: - -* Let them know if they need to sign a CLA, agree to a DCO, or get any other legal stuff out of the way -* If tests are required for contributions, let them know, and explain how to run the tests -* If you use anything other than GitHub to manage issues (ex. JIRA or Trac), let them know which tools they’ll need to contribute - ->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. - * Sign the Contributor License Agreement, CLA, with the jQuery Foundation. - * Note the jQuery Foundation Code of Conduct. - * Send a pull request indicating that you have a CLA on file. - -[source: [Requirejs](http://requirejs.org/docs/contributing.html)] **Need more inspiration?** [1] [Active Admin](https://github.com/activeadmin/activeadmin/blob/master/CONTRIBUTING.md#1-where-do-i-go-from-here) [2] [Node.js](https://github.com/nodejs/node/blob/master/CONTRIBUTING.md#code-contributions) [3] [Ember.js](https://github.com/emberjs/ember.js/blob/master/CONTRIBUTING.md#pull-requests) - -### If you have a different process for small or "obvious" fixes, let them know. - -> 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, without a CLA. - ->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 Gemfile, .gitignore, build scripts, etc. -* Moving source files from one directory or package to another - -[source: [Chef](https://github.com/chef/chef/blob/master/CONTRIBUTING.md#chef-obvious-fix-policy)] **Need more inspiration?** [1] [Puppet](https://github.com/puppetlabs/puppet/blob/master/CONTRIBUTING.md#making-trivial-changes) - -# 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 XXXX 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 security@travis-ci.org -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 security@travis-ci.org. - -[source: [Travis CI](https://github.com/travis-ci/travis-ci/blob/master/CONTRIBUTING.md)] **Need more inspiration?** [1] [Celery](https://github.com/celery/celery/blob/master/CONTRIBUTING.rst#security) [2] [Express.js](https://github.com/expressjs/express/blob/master/Security.md) - -### Tell your contributors how to 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 answer these five questions: - ->1. What version of Go are you using (go version)? -2. What operating system and processor architecture are you using? -3. What did you do? -4. What did you expect to see? -5. What did you see instead? -> General questions should go to the golang-nuts mailing list instead of the issue tracker. The gophers there will answer or ask you to file an issue if you've tripped over a bug. - -[source: [Go](https://github.com/golang/go/blob/master/CONTRIBUTING.md#filing-issues)] **Need more inspiration?** [1] [Celery](https://github.com/celery/celery/blob/master/CONTRIBUTING.rst#other-bugs ) [2] [Atom](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#reporting-bugs) (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](https://github.com/expressjs/express#philosophy)] **Need more inspiration?** [Active Admin](https://github.com/activeadmin/activeadmin#goals) - -### Explain your desired process for suggesting a feature. -If there is back-and-forth or signoff 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](https://github.com/elastic/elasticsearch/blob/master/CONTRIBUTING.md#feature-requests)] **Need more inspiration?** [1] [Hoodie](https://github.com/hoodiehq/hoodie/blob/master/CONTRIBUTING.md#feature-requests) [2] [Ember.js](https://github.com/emberjs/ember.js/blob/master/CONTRIBUTING.md#requesting-a-feature) - -# 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](https://github.com/puppetlabs/puppet/blob/master/CONTRIBUTING.md#submitting-changes)] **Need more inspiration?** [1] [Meteor](https://meteor.hackpad.com/Responding-to-GitHub-Issues-SKE2u3tkSiH ) [2] [Express.js](https://github.com/expressjs/express/blob/master/Contributing.md#becoming-a-committer) - -# 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](https://github.com/cucumber/cucumber-ruby/blob/master/CONTRIBUTING.md#talking-with-other-devs)] **Need more inspiration?** - [1] [Chef](https://github.com/chef/chef/blob/master/CONTRIBUTING.md#-developer-office-hours) [2] [Cookiecutter](https://github.com/audreyr/cookiecutter#community) - -# 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](http://requirejs.org/docs/contributing.html#codestyle) [2] [Elasticsearch](https://github.com/elastic/elasticsearch/blob/master/CONTRIBUTING.md#contributing-to-the-elasticsearch-codebase) - -### Explain if you use any commit message conventions. - -**Need inspiration?** [1] [Angular](https://github.com/angular/material/blob/master/CONTRIBUTING.md#-git-commit-guidelines) [2] [Node.js](https://github.com/nodejs/node/blob/master/CONTRIBUTING.md#step-3-commit) - -### Explain if you use any labeling conventions for issues. - -**Need inspiration?** [1] [StandardIssueLabels](https://github.com/wagenet/StandardIssueLabels#standardissuelabels) [2] [Atom](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#issue-and-pull-request-labels) +Sorry, we currently do not have any tags for truly beginner-friendly issues :( From ae37a4c85d183bcb496b0a7250fee7cbb728ed61 Mon Sep 17 00:00:00 2001 From: Sean Kaim Date: Wed, 30 Nov 2016 13:49:34 -0500 Subject: [PATCH 5/6] some additions - added short code style section - added "ready" and "Documentation" label notes - made note of Static Code Analysis tools for PRs --- .github/CONTRIBUTING.md | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 6d29bf61..81588181 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -45,8 +45,16 @@ Our requests when it comes to pull requests: - Keep your PR focused. It should be clear what code changed to achieve your stated goal. - Add or update tests when possible. Some of the code base is still very hard to effectively test. If you can, please try to create or update tests that relate to your PR. - Please talk with one of the project maintainers before starting work on large pull requests. Otherwise, you run the risk of putting a lot of time into a feature or refactor that gets denied. +- If you have access to Static Code Analysis tools (like ReSharper), please don't analyize/fix everything in one giant PR. As great as these tools are, they can some times inadvertantly cause an unintended problem. + +# Code Style +- Please make use of simple types and var where possilbe +- Prefer "using" over .Dispose() +- Avoid nesting "using" # Your first contributions -Want to help but don't know where to start? Check out the issues that we've labeled with [`Help Wanted`](https://github.com/mRemoteNG/mRemoteNG/issues?q=is%3Aissue+is%3Aopen+label%3A%22Help+Wanted%22). These will vary in difficulty, but should be possible for new contributors. +Want to help but don't know where to start? Check out the issues that we've labeled with [`Help Wanted`](https://github.com/mRemoteNG/mRemoteNG/issues?q=is%3Aissue+is%3Aopen+label%3A%22Help+Wanted%22) or [`ready`](https://github.com/mRemoteNG/mRemoteNG/issues?utf8=%E2%9C%93&q=is%3Aissue%20is%3Aopen%20label%3A%22ready%22). These will vary in difficulty, but should be possible for new contributors. + +Want to help but don't know C#? Check out the issues that we've labeled with [`Documentation`](https://github.com/mRemoteNG/mRemoteNG/labels/Documentation). Sorry, we currently do not have any tags for truly beginner-friendly issues :( From cd7ddb5ba1d2791400713f5be5a1da7767769656 Mon Sep 17 00:00:00 2001 From: David Sparer Date: Wed, 30 Nov 2016 12:08:48 -0700 Subject: [PATCH 6/6] minor typo fixes --- .github/CONTRIBUTING.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 81588181..d86b1b91 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -45,10 +45,10 @@ Our requests when it comes to pull requests: - Keep your PR focused. It should be clear what code changed to achieve your stated goal. - Add or update tests when possible. Some of the code base is still very hard to effectively test. If you can, please try to create or update tests that relate to your PR. - Please talk with one of the project maintainers before starting work on large pull requests. Otherwise, you run the risk of putting a lot of time into a feature or refactor that gets denied. -- If you have access to Static Code Analysis tools (like ReSharper), please don't analyize/fix everything in one giant PR. As great as these tools are, they can some times inadvertantly cause an unintended problem. +- If you have access to Static Code Analysis tools (like ReSharper), please don't analyze/fix everything in one giant PR. As great as these tools are, they can sometimes cause unintended problems. # Code Style -- Please make use of simple types and var where possilbe +- Please make use of simple types and var where possible - Prefer "using" over .Dispose() - Avoid nesting "using"