From 3ebfd6dde2ab1fb704ae046f959ea56b4bdbf1c4 Mon Sep 17 00:00:00 2001 From: Lee Dohm Date: Wed, 21 Dec 2016 19:58:39 -0800 Subject: [PATCH 1/3] Update issue and PR templates --- ISSUE_TEMPLATE.md | 36 ++++++++++++++++++++++++------------ PULL_REQUEST_TEMPLATE.md | 32 ++++++++++++++++++++++++++++++++ 2 files changed, 56 insertions(+), 12 deletions(-) create mode 100644 PULL_REQUEST_TEMPLATE.md diff --git a/ISSUE_TEMPLATE.md b/ISSUE_TEMPLATE.md index d2ac45f05..b60bb86c9 100644 --- a/ISSUE_TEMPLATE.md +++ b/ISSUE_TEMPLATE.md @@ -1,17 +1,23 @@ + + ### Prerequisites -* [ ] Can you reproduce the problem in [safe mode](http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode)? -* [ ] Are you running the [latest version of Atom](http://flight-manual.atom.io/hacking-atom/sections/debugging/#update-to-the-latest-version)? -* [ ] Did you check the [debugging guide](http://flight-manual.atom.io/hacking-atom/sections/debugging/)? -* [ ] Did you check the [FAQs on Discuss](https://discuss.atom.io/c/faq)? -* [ ] Are you reporting to the [correct repository](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#atom-and-packages)? -* [ ] Did you [perform a cursory search](https://github.com/issues?q=is%3Aissue+user%3Aatom+-repo%3Aatom%2Felectron) to see if your bug or enhancement is already reported? - -For more information on how to write a good [bug report](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#how-do-i-submit-a-good-bug-report) or [enhancement request](https://github.com/atom/atom/blob/master/CONTRIBUTING.md#how-do-i-submit-a-good-enhancement-suggestion), see the `CONTRIBUTING` guide. +* [ ] Put an X between the brackets on this line if you have done all of the following: + * Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode + * Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/ + * Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq + * Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom + * Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages ### Description -[Description of the bug or feature] +[Description of the issue] ### Steps to Reproduce @@ -19,10 +25,16 @@ For more information on how to write a good [bug report](https://github.com/atom 2. [Second Step] 3. [and so on...] -**Expected behavior:** [What you expected to happen] +**Expected behavior:** [What you expect to happen] -**Actual behavior:** [What actually happened] +**Actual behavior:** [What actually happens] + +**Reproduces how often:** [What percentage of the time does it reproduce?] ### Versions -You can get this information from executing `atom --version` and `apm --version` at the command line. Also, please include the OS and what version of the OS you're running. +You can get this information from copy and pasting the output of `atom --version` and `apm --version` from the command line. Also, please include the OS and what version of the OS you're running. + +### Additional Information + +Any additional information, configuration or data that might be necessary to reproduce the issue. diff --git a/PULL_REQUEST_TEMPLATE.md b/PULL_REQUEST_TEMPLATE.md new file mode 100644 index 000000000..c854f701e --- /dev/null +++ b/PULL_REQUEST_TEMPLATE.md @@ -0,0 +1,32 @@ +### Requirements + +* Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion. +* All new code requires tests to ensure against regressions + +### Description of the Change + + + +## Alternate Designs + + + +### [atom/atom only] Why Should This Be In Core? + + + +### Benefits + + + +### Possible Drawbacks + + + +### Applicable Issues + + From 821a54e18fe594aa7cc3353d23210e0e2aad792f Mon Sep 17 00:00:00 2001 From: Lee Dohm Date: Wed, 21 Dec 2016 20:12:22 -0800 Subject: [PATCH 2/3] Update CONTRIBUTING to reference the templates --- CONTRIBUTING.md | 86 ++++--------------------------------------------- 1 file changed, 6 insertions(+), 80 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ceb4186d9..e2e52e31a 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -84,7 +84,7 @@ When we make a significant decision in how we maintain the project and what we c This section guides you through submitting a bug report for Atom. Following these guidelines helps maintainers and the community understand your report :pencil:, reproduce the behavior :computer: :computer:, and find related reports :mag_right:. -Before creating bug reports, please check [this list](#before-submitting-a-bug-report) as you might find out that you don't need to create one. When you are creating a bug report, please [include as many details as possible](#how-do-i-submit-a-good-bug-report). If you'd like, you can use [this template](#template-for-submitting-bug-reports) to structure the information. +Before creating bug reports, please check [this list](#before-submitting-a-bug-report) as you might find out that you don't need to create one. When you are creating a bug report, please [include as many details as possible](#how-do-i-submit-a-good-bug-report). Fill out [the required template](ISSUE_TEMPLATE.md), the information it asks for helps us resolve issues faster. #### Before Submitting A Bug Report @@ -95,7 +95,7 @@ Before creating bug reports, please check [this list](#before-submitting-a-bug-r #### How Do I Submit A (Good) Bug Report? -Bugs are tracked as [GitHub issues](https://guides.github.com/features/issues/). After you've determined [which repository](#atom-and-packages) your bug is related to, create an issue on that repository and provide the following information. +Bugs are tracked as [GitHub issues](https://guides.github.com/features/issues/). After you've determined [which repository](#atom-and-packages) your bug is related to, create an issue on that repository and provide the following information by filling in [the template](ISSUE_TEMPLATE.md). Explain the problem and include additional details to help maintainers reproduce the problem: @@ -128,47 +128,11 @@ Include details about your configuration and environment: * **Are you using Atom with multiple monitors?** If so, can you reproduce the problem when you use a single monitor? * **Which keyboard layout are you using?** Are you using a US layout or some other layout? -#### Template For Submitting Bug Reports - - [Short description of problem here] - - **Reproduction Steps:** - - 1. [First Step] - 2. [Second Step] - 3. [Other Steps...] - - **Expected behavior:** - - [Describe expected behavior here] - - **Observed behavior:** - - [Describe observed behavior here] - - **Screenshots and GIFs** - - ![Screenshots and GIFs which follow reproduction steps to demonstrate the problem](url) - - **Atom version:** [Enter Atom version here] - **OS and version:** [Enter OS name and version here] - - **Installed packages:** - - [List of installed packages here] - - **Additional information:** - - * Problem can be reproduced in safe mode: [Yes/No] - * Problem started happening recently, didn't happen in an older version of Atom: [Yes/No] - * Problem can be reliably reproduced, doesn't happen randomly: [Yes/No] - * Problem happens with all files and projects, not only some files or projects: [Yes/No] - ### Suggesting Enhancements This section guides you through submitting an enhancement suggestion for Atom, including completely new features and minor improvements to existing functionality. Following these guidelines helps maintainers and the community understand your suggestion :pencil: and find related suggestions :mag_right:. -Before creating enhancement suggestions, please check [this list](#before-submitting-an-enhancement-suggestion) as you might find out that you don't need to create one. When you are creating an enhancement suggestion, please [include as many details as possible](#how-do-i-submit-a-good-enhancement-suggestion). If you'd like, you can use [this template](#template-for-submitting-enhancement-suggestions) to structure the information. +Before creating enhancement suggestions, please check [this list](#before-submitting-an-enhancement-suggestion) as you might find out that you don't need to create one. When you are creating an enhancement suggestion, please [include as many details as possible](#how-do-i-submit-a-good-enhancement-suggestion). Fill in [the template](ISSUE_TEMPLATE.md), including the steps that you imagine you would take if the feature you're requesting existed. #### Before Submitting An Enhancement Suggestion @@ -191,33 +155,6 @@ Enhancement suggestions are tracked as [GitHub issues](https://guides.github.com * **Specify which version of Atom you're using.** You can get the exact version by running `atom -v` in your terminal, or by starting Atom and running the `Application: About` command from the [Command Palette](https://github.com/atom/command-palette). * **Specify the name and version of the OS you're using.** -#### Template For Submitting Enhancement Suggestions - - [Short description of suggestion] - - **Steps which explain the enhancement** - - 1. [First Step] - 2. [Second Step] - 3. [Other Steps...] - - **Current and suggested behavior** - - [Describe current and suggested behavior here] - - **Why would the enhancement be useful to most users** - - [Explain why the enhancement would be useful to most users] - - [List some other text editors or applications where this enhancement exists] - - **Screenshots and GIFs** - - ![Screenshots and GIFs which demonstrate the steps or part of Atom the enhancement suggestion is related to](url) - - **Atom Version:** [Enter Atom version here] - **OS and Version:** [Enter OS name and version here] - ### Your First Code Contribution Unsure where to begin contributing to Atom? You can start by looking through these `beginner` and `help-wanted` issues: @@ -231,6 +168,7 @@ If you want to read about using Atom or developing packages in Atom, the [Atom F ### Pull Requests +* Fill in [the required template](PULL_REQUEST_TEMPLATE.md) * Include screenshots and animated GIFs in your pull request whenever possible. * Follow the [JavaScript](#javascript-styleguide) and [CoffeeScript](#coffeescript-styleguide) styleguides. * Include thoughtfully-worded, well-structured @@ -240,16 +178,12 @@ If you want to read about using Atom or developing packages in Atom, the [Atom F * End files with a newline. * Place requires in the following order: * Built in Node Modules (such as `path`) - * Built in Atom and Atom Shell Modules (such as `atom`, `shell`) + * Built in Atom and Electron Modules (such as `atom`, `remote`) * Local Modules (using relative paths) * Place class properties in the following order: * Class methods and properties (methods starting with a `@`) * Instance methods and properties -* Avoid platform-dependent code: - * Use `require('fs-plus').getHomeDirectory()` to get the home directory. - * Use `path.join()` to concatenate filenames. - * Use `os.tmpdir()` rather than `/tmp` when you need to reference the - temporary directory. +* [Avoid platform-dependent code](http://flight-manual.atom.io/hacking-atom/sections/cross-platform-compatibility/) * Using a plain `return` when returning explicitly at the end of a function. * Not `return null`, `return undefined`, `null`, or `undefined` @@ -427,12 +361,6 @@ Please open an issue on `atom/atom` if you have suggestions for new labels, and | `deprecation-help` | [search][search-atom-repo-label-deprecation-help] | [search][search-atom-org-label-deprecation-help] | Issues for helping package authors remove usage of deprecated APIs in packages. | | `electron` | [search][search-atom-repo-label-electron] | [search][search-atom-org-label-electron] | Issues that require changes to [Electron](https://electron.atom.io) to fix or implement. | -#### Core Team Project Management - -| Label name | `atom/atom` :mag_right: | `atom`‑org :mag_right: | Description | -| --- | --- | --- | --- | -| `atom` | [search][search-atom-repo-label-atom] | [search][search-atom-org-label-atom] | Topics discussed for prioritization at the next meeting of Atom core team members. | - #### Pull Request Labels | Label name | `atom/atom` :mag_right: | `atom`‑org :mag_right: | Description @@ -519,8 +447,6 @@ Please open an issue on `atom/atom` if you have suggestions for new labels, and [search-atom-org-label-deprecation-help]: https://github.com/issues?q=is%3Aopen+is%3Aissue+user%3Aatom+label%3Adeprecation-help [search-atom-repo-label-electron]: https://github.com/issues?q=is%3Aissue+repo%3Aatom%2Fatom+is%3Aopen+label%3Aelectron [search-atom-org-label-electron]: https://github.com/issues?q=is%3Aopen+is%3Aissue+user%3Aatom+label%3Aelectron -[search-atom-repo-label-atom]: https://github.com/issues?q=is%3Aopen+is%3Aissue+repo%3Aatom%2Fatom+label%3Aatom -[search-atom-org-label-atom]: https://github.com/issues?q=is%3Aopen+is%3Aissue+user%3Aatom+label%3Aatom [search-atom-repo-label-work-in-progress]: https://github.com/pulls?q=is%3Aopen+is%3Apr+repo%3Aatom%2Fatom+label%3Awork-in-progress [search-atom-org-label-work-in-progress]: https://github.com/pulls?q=is%3Aopen+is%3Apr+user%3Aatom+label%3Awork-in-progress [search-atom-repo-label-needs-review]: https://github.com/pulls?q=is%3Aopen+is%3Apr+repo%3Aatom%2Fatom+label%3Aneeds-review From 23696fd82a2cf24aa907c7b8e9e9f7334c4212c3 Mon Sep 17 00:00:00 2001 From: Lee Dohm Date: Wed, 21 Dec 2016 20:15:05 -0800 Subject: [PATCH 3/3] Remove atom/atom only reference --- PULL_REQUEST_TEMPLATE.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PULL_REQUEST_TEMPLATE.md b/PULL_REQUEST_TEMPLATE.md index c854f701e..a617961db 100644 --- a/PULL_REQUEST_TEMPLATE.md +++ b/PULL_REQUEST_TEMPLATE.md @@ -15,7 +15,7 @@ We must be able to understand the design of your change from this description. I -### [atom/atom only] Why Should This Be In Core? +### Why Should This Be In Core?