From 887465f4ae63d197416bd91c39ff3d0eaa8d67f8 Mon Sep 17 00:00:00 2001 From: alexweissman Date: Mon, 23 Oct 2017 15:11:39 -0400 Subject: [PATCH] update contributing, issue template docs --- .github/CONTRIBUTING.md | 149 +++++++++++++++++++------------------- .github/ISSUE_TEMPLATE.md | 54 ++++---------- 2 files changed, 88 insertions(+), 115 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index c4d00696..a9a5733b 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -1,64 +1,52 @@ -Contributing to Select2 -======================= -Looking to contribute something to Select2? **Here's how you can help.** +# Guidelines for Getting Help with Select2 -Please take a moment to review this document in order to make the contribution -process easy and effective for everyone involved. +**Before** you open a new issue, you must **read these guidelines.** If it is evident from your issue that you failed to research your question properly, **your issue may be closed** without being answered. -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 or assessing -patches and features. +## Troubleshooting -Using the issue tracker ------------------------ -When [reporting bugs][reporting-bugs] or -[requesting features][requesting-features], the -[issue tracker on GitHub][issue-tracker] is the recommended channel to use. +There are some common problems that developers may encounter when using working with Select2 for the first time. **If you are new to Select2**, please consult the [common problems](https://select2.org/troubleshooting/common-problems) section of the documentation first. -The issue tracker **is not** a place for support requests. The -[mailing list][community] or [IRC channel][community] are better places to -get help. +If you don't find what you're looking for there, then please check the the [forums](https://forums.select2.org), [Stack Overflow](https://stackoverflow.com/questions/tagged/jquery-select2) and [existing issues](https://github.com/select2/select2/issues?utf8=%E2%9C%93&q=is%3Aissue), both opened and closed. Your question may have already been asked and answered before! -Reporting bugs with Select2 ---------------------------- -We really appreciate clear bug reports that _consistently_ show an issue -_within Select2_. +## Asking for Help -The ideal bug report follows these guidelines: +In general, the Github issue tracker should **only** be used for **bug reports** and **feature requests**. If you're just having trouble getting something to work, you should ask in the [forums](https://forums.select2.org) or on [Stack Overflow](https://stackoverflow.com) instead. -1. **Use the [GitHub issue search][issue-search]** — Check if the issue - has already been reported. -2. **Check if the issue has been fixed** — Try to reproduce the problem - using the code in the `master` branch. -3. **Isolate the problem** — Try to create an - [isolated test case][isolated-case] that consistently reproduces the problem. +### Rules for all community platforms -Please try to be as detailed as possible in your bug report, especially if an -isolated test case cannot be made. Some useful questions to include the answer -to are: +1. Be polite, civil and respectful. Select2 is built and maintained entirely by volunteers. +2. If you are not conversationally proficient in English, **do not just post a machine translation** (e.g. Google Translate). Post in your native language, so that others who speak your language can help. You may post a machine translation below it for the general community to decipher. +3. Any code snippets should be formatted using [Markdown code fences](https://learn.userfrosting.com/troubleshooting/getting-help#use-markdown-to-format-blocks-of-code) and properly indented. Poorly formatted code is difficult to read and reflects badly on you as a programmer. +4. Check what you write for spelling and grammar issues. If you want others to take the time to read your question carefully, you must write your question carefully. +5. When possible create and link to a [minimal, complete, and verifiable](https://stackoverflow.com/help/mcve) example by cloning our [JSBin template](http://jsbin.com/goqiqolete/edit?html,js,output). Code dumps, zip files, etc are NOT acceptable. +6. Include relevant screenshots or animations, if possible. Drag your screenshots directly into the forums or issue tracker text box. They will automatically be uploaded and the Markdown to display them will be generated. Avoid third-party image hosts, or links which require extra clicks to view the image (except IRC, where this is unavoidable). -- What steps can be used to reproduce the issue? -- What is the bug and what is the expected outcome? -- What browser(s) and Operating System have you tested with? -- Does the bug happen consistently across all tested browsers? -- What version of jQuery are you using? And what version of Select2? -- Are you using Select2 with other plugins? +### Stack Overflow -All of these questions will help others fix and identify any potential bugs. +Tag your question with the `jquery-select2` tag, and optionally with tags relevant to other technologies involved, such as `jquery` or another client-side framework. You should also mention the version of Select2 that you are using. + +### GitHub Issue Tracker + +We really appreciate clear bug reports that _consistently_ show an issue _within Select2_. If you are reporting a bug, you **must** follow these steps: + +1. Make sure that your issue is a bug or feature request. General usage and troubleshooting questions should be directed to the [forums](https://forums.select2.org) or [Stack Overflow](https://stackoverflow.com/questions/tagged/jquery-select2). Issues asking for general support WILL BE CLOSED automatically. +2. Search the current issues, both open and closed, for a similar issue. If the bug is still present but the relevant issue has been closed, you may ask us to reopen the issue. Duplicate issues will be closed automatically. +3. Make sure that you are using the latest stable version of Select2 (see the [release history](https://github.com/select2/select2/releases)). Old minor/patch versions will not be supported. +4. State the steps needed to reproduce the problem. +5. Report any errors in detail. Vague issues like "it doesn't work when I do this" are not helpful. Show that you have put some effort into identifying the cause of the error. Check your [browser console](https://learn.userfrosting.com/troubleshooting/debugging#client-side-debugging) for any Javascript error messages. +6. Mention your version of Select2, as well as the browser(s) and operating system(s) in which the problem occurs. Requesting features in Select2 ------------------------------ Select2 is a large library that carries with it a lot of functionality. Because of this, many feature requests will not be implemented in the core library. -Before starting work on a major feature for Select2, **contact the -[community][community] first** or you may risk spending a considerable amount of +Before starting work on a major feature for Select2, **post to the [forums](https://forums.select2.org) first** or you may risk spending a considerable amount of time on something which the project developers are not interested in bringing into the project. -Contributing changes to Select2 -------------------------------- +# Contributing to Select2 + Select2 is made up of multiple submodules that all come together to make the standard and extended builds that are available to users. The build system uses Node.js to manage and compile the submodules, all of which is done using the @@ -66,25 +54,17 @@ Grunt build system. ### Installing development dependencies -Select2 can be built and developed on any system which supports Node.js. The -preferred Node.js version is 0.10, but 0.12 and later versions can be used -without any noticeable issues. You can download Node.js at +Select2 can be built and developed on any system which supports Node.js. You can download Node.js at [their website][nodejs]. -All other required Node.js packages can be installed using [npm][npm], which -comes bundled alongside Node.js. +All other required Node.js packages can be installed using [npm][npm], which comes bundled alongside Node.js. ```bash cd /path/to/select2/repo npm install ``` -You may need to install libsass on your system if it is not already available -in order to build the SASS files which generate the CSS for themes and the main -component. - -In order to build and serve the documentation, you need to have [Jekyll][jekyll] -installed on your system. +You may need to install `libsass` on your system if it is not already available in order to build the SASS files which generate the CSS for themes and the main component. ### Building the Select2 component @@ -105,17 +85,6 @@ cd /path/to/select2/repo grunt minify ``` -### Building the documentation - -Using the Grunt build system, you run Jekyll and serve the documentation -locally. This will also set up the examples to use the latest version of -Select2 that has been built. - -```bash -cd /path/to/select2/repo -grunt docs -``` - ### Running tests Select2 uses the QUnit test system to test individual components. @@ -125,6 +94,45 @@ cd /path/to/selct2/repo grunt test ``` +### Setting up the documentation repo + +The documentation for Select2 is maintained in a [separate repository](https://github.com/select2/docs). Select2.org is built with the flat-file CMS [Grav](http://getgrav.org), using their [RTFM skeleton](https://github.com/getgrav/grav-skeleton-rtfm-site#rtfm-skeleton). + +The documentation files themselves are written in Markdown, and can be found in the `pages/` subdirectory. You can submit pull requests to the `develop` branch of the repo. + +If you'd like to set up a local instance of the entire documentation website, you must first have a web server (Nginx, Apache, etc) and PHP installed locally. Then, follow these steps: + +#### Step 1 - Install Grav + +This application uses the [Grav](https://learn.getgrav.org/) CMS. This repository does not contain a full Grav installation - rather, it just contains the contents of Grav's `user` directory, which is where all of our content, themes, and assets live. This was done as per the [recommendation on Grav's blog](https://getgrav.org/blog/developing-with-github-part-2), to make it easier to deploy changes to the live server. + +To install this website on your computer, first [install grav core](https://getgrav.org/downloads) in a project folder called `select2-docs` under your webserver's document root folder. Then, find the `user` folder inside of your project folder. Delete the contents of the `user` folder and clone this repository directly into the user folder. + +When you're done it might look something like this: + +``` +htdocs/ +└── select2-docs/ + ├── assets/ + ├── ... + ├── user/ + ├── .git + ├── accounts/ + ├── assets/ + ├── config/ + └── ... + └── ... + +``` + +#### Step 2 + +Grav needs your webserver to be able to write to certain directories. Make sure that `backup/`, `cache/`, `images/`, `logs/`, and `tmp/` are all writeable by the user account under which your webserver runs. + +#### Step 3 + +Visit the local installation in your browser! For example, [http://localhost/select2-docs](http://localhost/select2-docs). + ### Submitting a pull request We use GitHub's pull request system for submitting patches. Here are some @@ -141,9 +149,7 @@ pull request will be familiar with the problem it is fixing. 4. Run your tests first. If your tests aren't passing, the pull request won't be able to be merged. If you're breaking existing tests, make sure that you aren't causing any breaking changes. -5. Only include source changes. While it's not required, only including changes -from the `src` directory will prevent merge conflicts from occurring. Making -this happen can be as a simple as not committing changes from the `dist` +5. Only include source changes (`src/`). Do not make changes directly to files in the `dist` directory. By following these steps, you will make it easier for your pull request to be @@ -178,10 +184,7 @@ a comment on the pull request. ### Reviving old tickets -If you come across tickets which have not been updated for a while, you are -encouraged to revive them. While this can be as simple as saying `:+1:`, it is -best if you can include more information on the issue. Common bugs and feature -requests are more likely to be fixed, whether it is by the community or the +If you come across tickets which have not been updated for a while, you are encouraged to revive them. If you do, please include more information in your comment on the issue. Common bugs and feature requests are more likely to be fixed, whether it is by the community or the developers, so keeping tickets up to date is encouraged. Licensing @@ -191,14 +194,10 @@ It should also be made clear that **all code contributed to Select** must be licensable under the [MIT license][licensing]. Code that cannot be released under this license **cannot be accepted** into the project. -[community]: https://select2.github.io/community.html [grunt]: http://gruntjs.com/ [isolated-case]: http://css-tricks.com/6263-reduced-test-cases/ [issue-search]: https://github.com/select2/select2/search?q=&type=Issues [issue-tracker]: https://github.com/select2/select2/issues -[jekyll]: https://jekyllrb.com/docs/installation/ [licensing]: https://github.com/select2/select2/blob/master/LICENSE.md [nodejs]: https://nodejs.org/ [npm]: https://www.npmjs.com/ -[reporting-bugs]: #reporting-bugs-with-select2 -[requesting-features]: #requesting-features-in-select2 diff --git a/.github/ISSUE_TEMPLATE.md b/.github/ISSUE_TEMPLATE.md index 83338844..0359d57d 100644 --- a/.github/ISSUE_TEMPLATE.md +++ b/.github/ISSUE_TEMPLATE.md @@ -1,46 +1,20 @@ -## Prerequisites -- [ ] I have searched for similar issues in both open and closed tickets and cannot find a duplicate -- [ ] The issue still exists against the latest `master` branch of Select2 -- [ ] This is not a usage question (Those should be directed to the [community](https://select2.github.io/community.html)) -- [ ] I have attempted to find the simplest possible steps to reproduce the issue -- [ ] I have included a failing test as a pull request (Optional) +STOP! -## Steps to reproduce the issue +Before you open an issue: -1. -2. -3. +1. Make sure that your issue is a BUG or FEATURE REQUEST. General usage and troubleshooting questions should be directed to the [forums](https://forums.select2.org) or [Stack Overflow](https://stackoverflow.com/questions/tagged/jquery-select2). Issues asking for general support WILL BE CLOSED automatically. +2. Search the current issues, both open and closed, for a similar issue. If the bug is still present but the relevant issue has been closed, you may ask us to reopen the issue. Duplicate issues will be closed automatically. +3. Make sure that you are using the latest stable version of Select2 (see the [release history](https://github.com/select2/select2/releases)). Old minor/patch versions will not be supported. -## Expected behavior and actual behavior +If you are CERTAIN that it is appropriate to open a new issue, you must: -When I follow those steps, I see... +1. Create and link to a [minimal, complete, and verifiable](https://stackoverflow.com/help/mcve) example by cloning our [JSBin template](http://jsbin.com/goqiqolete/edit?html,js,output). Code dumps, zip files, etc are NOT acceptable. +2. Any additional code snippets should be formatted using [Markdown code fences](https://learn.userfrosting.com/troubleshooting/getting-help#use-markdown-to-format-blocks-of-code) and properly indented. Poorly formatted code is difficult to read and reflects badly on you as a programmer. +3. Include relevant screenshots or animations, if possible. Drag your screenshots directly into this text box. They will automatically be uploaded and the Markdown to display them will be generated. Avoid third-party image hosts, or links which require extra clicks to view the image. +4. If you are not conversationally proficient in English, do NOT just post a machine translation (e.g. Google Translate) to GitHub. Get help in crafting your question, either via the [forums](https://forums.select2.org) or in [chat](https://webchat.freenode.net/?channels=select2). If all else fails, you may post your bug report or feature request in your native language, with a machine translation below that. We will tag it with `translation-needed` so that others who speak your language can find it. +5. State the steps needed to reproduce the problem. +6. Mention your version of Select2, as well as the browser(s) and operating system(s) in which the problem occurs. +7. Clear out this placeholder text. -I was expecting... - -## Environment - -Browsers - -- [ ] Google Chrome -- [ ] Mozilla Firefox -- [ ] Internet Explorer - -Operating System - -- [ ] Windows -- [ ] Mac OS X -- [ ] Linux -- [ ] Mobile - -Libraries - -- jQuery version: -- Select2 version: - -## Isolating the problem - -- [ ] This bug happens [on the examples page](https://select2.github.io/examples.html) -- [ ] The bug happens consistently across all tested browsers -- [ ] This bug happens when using Select2 without other plugins -- [ ] I can reproduce this bug in [a jsbin](https://jsbin.com/) +Issues that ignore these rules will be closed without further comment!