Browse Source

Add templates, contributing, code of conduct

Scott Lahteine 6 years ago
parent
commit
4d0584a828
4 changed files with 240 additions and 36 deletions
  1. 46
    0
      .github/code_of_conduct.md
  2. 144
    0
      .github/contributing.md
  3. 31
    36
      .github/issue_template.md
  4. 19
    0
      .github/pull_request_template.md

+ 46
- 0
.github/code_of_conduct.md View File

@@ -0,0 +1,46 @@
1
+# Contributor Covenant Code of Conduct
2
+
3
+## Our Pledge
4
+
5
+In the interest of fostering an open and welcoming environment, we as contributors and maintainers pledge to making participation in our project and our community a harassment-free experience for everyone, regardless of age, body size, disability, ethnicity, gender identity and expression, level of experience, nationality, personal appearance, race, religion, or sexual identity and orientation.
6
+
7
+## Our Standards
8
+
9
+Examples of behavior that contributes to creating a positive environment include:
10
+
11
+* Using welcoming and inclusive language
12
+* Being respectful of differing viewpoints and experiences
13
+* Gracefully accepting constructive criticism
14
+* Focusing on what is best for the community
15
+* Showing empathy towards other community members
16
+
17
+Examples of unacceptable behavior by participants include:
18
+
19
+* The use of sexualized language or imagery and unwelcome sexual attention or advances
20
+* Trolling, insulting/derogatory comments, and personal or political attacks
21
+* Public or private harassment
22
+* Publishing others' private information, such as a physical or electronic address, without explicit permission
23
+* Other conduct which could reasonably be considered inappropriate in a professional setting
24
+
25
+## Our Responsibilities
26
+
27
+Project maintainers are responsible for clarifying the standards of acceptable behavior and are expected to take appropriate and fair corrective action in response to any instances of unacceptable behavior.
28
+
29
+Project maintainers have the right and responsibility to remove, edit, or reject comments, commits, code, issues, and other contributions that are not aligned to this Code of Conduct, or to ban temporarily or permanently any contributor for other behaviors that they deem inappropriate, threatening, offensive, or harmful.
30
+
31
+## Scope
32
+
33
+This Code of Conduct applies both within project spaces and in public spaces when an individual is representing the project or its community. Examples of representing a project or community include using an official project e-mail address, posting via an official social media account, or acting as an appointed representative at an online or offline event. Representation of a project may be further defined and clarified by project maintainers.
34
+
35
+## Enforcement
36
+
37
+Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting the project team at [marlinfirmware@github.com](mailto:marlinfirmware@github.com). All complaints will be reviewed and investigated and will result in a response that is deemed necessary and appropriate to the circumstances. The project team is obligated to maintain confidentiality with regard to the reporter of an incident. Further details of specific enforcement policies may be posted separately.
38
+
39
+Project maintainers who do not follow or enforce the Code of Conduct in good faith may face temporary or permanent repercussions as determined by other members of the project's leadership.
40
+
41
+## Attribution
42
+
43
+This Code of Conduct is adapted from the [Contributor Covenant][homepage], version 1.4, available at [https://contributor-covenant.org/version/1/4][version]
44
+
45
+[homepage]: https://contributor-covenant.org
46
+[version]: https://contributor-covenant.org/version/1/4/

+ 144
- 0
.github/contributing.md View File

@@ -0,0 +1,144 @@
1
+# Contributing to Marlin
2
+
3
+Thanks for your interest in contributing to Marlin Firmware!
4
+
5
+The following is a set of guidelines for contributing to Marlin, hosted by the [MarlinFirmware Organization](https://github.com/MarlinFirmware) on GitHub. These are mostly guidelines, not rules. Use your best judgment, and feel free to propose changes to this document in a Pull Request.
6
+
7
+#### Table Of Contents
8
+
9
+[Code of Conduct](#code-of-conduct)
10
+
11
+[I don't want to read this whole thing, I just have a question!!!](#i-dont-want-to-read-this-whole-thing-i-just-have-a-question)
12
+
13
+[How Can I Contribute?](#how-can-i-contribute)
14
+  * [Reporting Bugs](#reporting-bugs)
15
+  * [Suggesting Features or Changes](#suggesting-features-or-changes)
16
+  * [Your First Code Contribution](#your-first-code-contribution)
17
+  * [Pull Requests](#pull-requests)
18
+
19
+[Styleguides](#styleguides)
20
+  * [Git Commit Messages](#git-commit-messages)
21
+  * [C++ Coding Standards](#c++-coding-standards)
22
+  * [Documentation Styleguide](#documentation)
23
+
24
+[Additional Notes](#additional-notes)
25
+  * [Issue and Pull Request Labels](#issue-and-pull-request-labels)
26
+
27
+## Code of Conduct
28
+
29
+This project and everyone participating in it is governed by the [Marlin Code of Conduct](code_of_conduct.md). By participating, you are expected to uphold this code. Please report unacceptable behavior to [marlinfirmware@github.com](mailto:marlinfirmware@github.com).
30
+
31
+## I don't want to read this whole thing I just have a question!!!
32
+
33
+> **Note:** Please don't file an issue to ask a question. You'll get faster results by using the resources below.
34
+
35
+We have a Message Board and a Facebook group where our knowledgable user community can provide helpful advice if you have questions.
36
+
37
+* [Marlin RepRap forum](http://forums.reprap.org/list.php?415)
38
+* [MarlinFirmware on Facebook](https://www.facebook.com/groups/1049718498464482/)
39
+
40
+If chat is more your speed, you can join the MarlinFirmware Slack team:
41
+
42
+* Join the Marlin Slack Team
43
+  * To obtain group access, please [send a request](http://www.thinkyhead.com/contact/9) to @thinkyhead.
44
+  * Even though Slack is a chat service, sometimes it takes several hours for community members to respond — please be patient!
45
+  * Use the `#general` channel for general questions or discussion about Marlin.
46
+  * Other channels exist for certain topics. Check the channel list.
47
+
48
+## How Can I Contribute?
49
+
50
+### Reporting Bugs
51
+
52
+This section guides you through submitting a Bug Report for Marlin. Following these guidelines helps maintainers and the community understand your report, reproduce the behavior, and find related reports.
53
+
54
+Before creating a Bug Report, please test the "nightly" development branch, 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.
55
+
56
+> **Note:** Regressions can happen. If you find a **Closed** issue that seems like your issue, go ahead and open a new issue and include a link to the original issue in the body of your new one. All you need to create a link is the issue number, preceded by #. For example, #8888.
57
+
58
+#### How Do I Submit A (Good) Bug Report?
59
+
60
+Bugs are tracked as [GitHub issues](https://guides.github.com/features/issues/). Use the New Issue button to create an issue and provide the following information by filling in [the template](issue_template.md).
61
+
62
+Explain the problem and include additional details to help maintainers reproduce the problem:
63
+
64
+* **Use a clear and descriptive title** for the issue to identify the problem.
65
+* **Describe the exact steps which reproduce the problem** in as many details as possible. For example, start by explaining how you started Marlin, e.g. which command exactly you used in the terminal, or how you started Marlin otherwise. When listing steps, **don't just say what you did, but explain how you did it**. For example, if you moved the cursor to the end of a line, explain if you used the mouse, or a keyboard shortcut or an Marlin command, and if so which one?
66
+* **Provide specific examples to demonstrate the steps**. Include links to files or GitHub projects, or copy/pasteable snippets, which you use in those examples. If you're providing snippets or log output in the issue, use [Markdown code blocks](https://help.github.com/articles/markdown-basics/#multiple-lines).
67
+* **Describe the behavior you observed after following the steps** and point out what exactly is the problem with that behavior.
68
+* **Explain which behavior you expected to see instead and why.**
69
+* **Include detailed log output** especially for probing and leveling. See below for usage of `DEBUG_LEVELING_FEATURE`.
70
+* **Include screenshots, links to videos, etc.** which clearly demonstrate the problem.
71
+* **Include G-code** (if relevant) that reliably causes the problem to show itself.
72
+* **If the problem wasn't triggered by a specific action**, describe what you were doing before the problem happened and share more information using the guidelines below.
73
+
74
+Provide more context:
75
+
76
+* **Can you reproduce the problem with a minimum of options enabled?**
77
+* **Did the problem start happening recently** (e.g. after updating to a new version of Marlin) or was this always a problem?
78
+* If the problem started happening recently, **can you reproduce the problem in an older version of Marlin?** What's the most recent version in which the problem doesn't happen? You can download older versions of Marlin from [the releases page](https://github.com/MarlinFirmware/Marlin/releases).
79
+* **Can you reliably reproduce the issue?** If not, provide details about how often the problem happens and under which conditions it normally happens.
80
+
81
+Include details about your configuration and environment:
82
+
83
+* **Which version of Marlin are you using?** Marlin's exact version and build date can be seen in the startup message when a host connects to Marlin, or in the LCD Info menu (if enabled).
84
+* **What kind of 3D Printer and electronics are you using**?
85
+* **What kind of add-ons (probe, filament sensor) do you have**?
86
+* **Include your Configuration files.** Make a ZIP file containing `Configuration.h` and `Configuration_adv.h` and drop it on your reply.
87
+
88
+### Suggesting Features or Changes
89
+
90
+This section guides you through submitting a suggestion for Marlin, including completely new features and minor improvements to existing functionality. Following these guidelines helps maintainers and the community understand your suggestion and find related suggestions.
91
+
92
+Before creating a suggestion, please check [this list](#before-submitting-a-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.
93
+
94
+#### Before Submitting a Feature Request
95
+
96
+* **Check the [Marlin website](http://marlinfw.org/)** for tips — you might discover that the feature is already included. Most importantly, check if you're using [the latest version of Marlin](https://github.com/MarlinFirmware/Marlin/releases) and if you can get the desired behavior by changing [Marlin's config settings](http://marlinfw.org/docs/configuration/configuration.html).
97
+* **Perform a [cursory search](https://github.com/MarlinFirmware/Marlin/issues?q=is%3Aissue)** to see if the enhancement has already been suggested. If it has, add a comment to the existing issue instead of opening a new one.
98
+
99
+#### How Do I Submit A (Good) Feature Request?
100
+
101
+Feature Requests are tracked as [GitHub issues](https://guides.github.com/features/issues/). Please follow these guidelines in your request:
102
+
103
+* **Use a clear and descriptive title** for the issue to identify the suggestion.
104
+* **Provide a step-by-step description of the requested feature** in as much detail as possible.
105
+* **Provide specific examples to demonstrate the steps**.
106
+* **Describe the current behavior** and **explain which behavior you expected to see instead** and why.
107
+* **Include screenshots and links to videos** which demonstrate the feature or point out the part of Marlin to which the request is related.
108
+* **Explain why this feature would be useful** to most Marlin users.
109
+* **Name other firmwares that have this feature, if any.**
110
+
111
+### Your First Code Contribution
112
+
113
+Unsure where to begin contributing to Marlin? You can start by looking through these `good-first-issue` and `help-wanted` issues:
114
+
115
+* [Beginner issues][good-first-issue] - issues which should only require a few lines of code, and a test or two.
116
+* [Help Wanted issues][help-wanted] - issues which should be a bit more involved than `beginner` issues.
117
+
118
+### Pull Requests
119
+
120
+Pull Requests should always be targeted to working branches (e.g., `bugfix-1.1.x` and/or `bugfix-2.0.x`) and never to release branches (e.g., `1.1.x`). If this is your first Pull Request, please read our [Guide to Pull Requests](http://marlinfw.org/docs/development/getting_started_pull_requests.html) and Github's [Pull Request](https://help.github.com/articles/creating-a-pull-request/) documentation.
121
+
122
+* Fill in [the required template](pull_request_template.md).
123
+* Don't include issue numbers in the PR title.
124
+* Include pictures, diagrams, and links to videos in your Pull Request to demonstrate your changes, if needed.
125
+* Follow the [Coding Standards](http://marlinfw.org/docs/development/coding_standards.html) posted on our website.
126
+* Document new code with clear and concise comments.
127
+* End all files with a newline.
128
+
129
+## Styleguides
130
+
131
+### Git Commit Messages
132
+
133
+* Use the present tense ("Add feature" not "Added feature").
134
+* Use the imperative mood ("Move cursor to..." not "Moves cursor to...").
135
+* Limit the first line to 72 characters or fewer.
136
+* Reference issues and Pull Requests liberally after the first line.
137
+
138
+### C++ Coding Standards
139
+
140
+* Please read and follow the [Coding Standards](http://marlinfw.org/docs/development/coding_standards.html) posted on our website. Failure to follow these guidelines will delay evaluation and acceptance of Pull Requests.
141
+
142
+### Documentation
143
+
144
+* Guidelines for documentation are still under development. In-general, be clear, concise, and to-the-point.

+ 31
- 36
.github/issue_template.md View File

@@ -1,36 +1,31 @@
1
-Thank you for submitting your feedback to the Marlin project.
2
-Please use one of the templates below to fill out this box.
3
-
4
-------------------------------------------------------------
5
-### Feature Request
6
-Please put [FR] in the issue title: `[FR] Add-on that goes 'ping'`
7
-
8
-------------------------------------------------------------
9
-### Compile Error
10
-When I compile with `FEATURE_X` I get an error:
11
-```
12
-Paste_the_error_text_here
13
-```
14
-
15
-------------------------------------------------------------
16
-### Bug Report
17
-- Description: ---
18
-- Expected behaviour: ---
19
-- Actual behaviour: ---
20
-- Steps to reproduce:
21
-  - Do this
22
-  - Do that
23
-
24
-Attach a ZIP of `Configuration.h` and `Configuration_adv.h` by dropping here.
25
-
26
-------------------------------------------------------------
27
-### Bug Report Tips
28
-- When troubleshooting, use `M502` followed by `M500` to reset EEPROM to defaults.
29
-- Use `DEBUG_LEVELING_FEATURE` with `M111 S247` for detailed logging of homing/leveling.
30
-- Format text with: **bold**, _italic_, `code`.
31
-- Format C++ with three backticks, plus "cpp":
32
-```cpp
33
-void my_function(bool do_it) {
34
-  // Hold this spot
35
-}
36
-```
1
+<!--
2
+
3
+Have you read Marlin's Code of Conduct? By filing an Issue, you are expected to comply with it, including treating everyone with respect: https://github.com/MarlinFirmware/Marlin/blob/master/.github/code_of_conduct.md
4
+
5
+Do you want to ask a question? Are you looking for support? Please don't post here. Instead please use the Marlin Firmware forum at http://forums.reprap.org/list.php?415 or the Marlin Facebook Group https://www.facebook.com/groups/1049718498464482/.
6
+
7
+Before filing an issue be sure to test the 1.1 and/or 2.0 "bugfix" branches to see whether the issue is already addressed.
8
+
9
+-->
10
+
11
+### Description
12
+
13
+<!-- Description of the bug or requested feature -->
14
+
15
+### Steps to Reproduce
16
+
17
+<!-- If this is a Bug Report, please describe the steps needed to reproduce the issue -->
18
+
19
+1. [First Step]
20
+2. [Second Step]
21
+3. [and so on...]
22
+
23
+**Expected behavior:** [What you expect to happen]
24
+
25
+**Actual behavior:** [What actually happens]
26
+
27
+#### Additional Information
28
+
29
+* Include a ZIP file containing your `Configuration.h` and `Configuration_adv.h` files.
30
+* Provide pictures or links to videos that clearly demonstrate the issue.
31
+* See [How Can I Contribute](#how-can-i-contribute) for additional guidelines.

+ 19
- 0
.github/pull_request_template.md View File

@@ -0,0 +1,19 @@
1
+### Requirements
2
+
3
+* Filling out this template is required. Pull Requests without a clear description may be closed at the maintainers' discretion.
4
+
5
+### Description
6
+
7
+<!--
8
+
9
+We must be able to understand your proposed change from this description. If we can't understand what the code will do from this description, the Pull Request may be closed at the maintainers' discretion. Keep in mind that the maintainer reviewing this PR may not be familiar with or have worked with the code recently, so please walk us through the concepts.
10
+
11
+-->
12
+
13
+### Benefits
14
+
15
+<!-- What does this fix or improve? -->
16
+
17
+### Related Issues
18
+
19
+<!-- Whether this fixes a bug or fulfills a feature request, please list any related Issues here. -->

Loading…
Cancel
Save