You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Nov 19, 2024. It is now read-only.
Copy file name to clipboardExpand all lines: .github/CONTRIBUTING.md
+33-33Lines changed: 33 additions & 33 deletions
Original file line number
Diff line number
Diff line change
@@ -58,33 +58,33 @@ The following diagram shows the contribution workflow:
58
58
59
59
The following guidelines may answer most of your questions and help you get started:
60
60
61
-
- Write content using Markdown. See the [Templates](#templates) section for examples.
62
-
- Review existing [pull requests](https://github.com/magento/devdocs/pulls) and [issues](https://github.com/magento/devdocs/issues) to avoid duplicating work.
63
-
- For large contributions, or changes that include multiple files, [open an issue](#report-an-issue) and discuss it with us first. This helps prevent duplicate or unnecessary work.
64
-
- Do not make global find-and-replace changes without first creating an issue and discussing it with us. Global changes can have unintended consequences.
65
-
- Combine multiple small changes (such as minor editorial and technical changes) into a single pull request. This helps us efficiently and effectively facilitate your contribution.
66
-
- Familiarize yourself with the organization and conventions of our existing documentation before creating a pull request. Changes that are consistent with our style and conventions have a higher acceptance rate.
61
+
-Write content using Markdown. See the [Templates](#templates) section for examples.
62
+
-Review existing [pull requests](https://github.com/magento/devdocs/pulls) and [issues](https://github.com/magento/devdocs/issues) to avoid duplicating work.
63
+
-For large contributions, or changes that include multiple files, [open an issue](#report-an-issue) and discuss it with us first. This helps prevent duplicate or unnecessary work.
64
+
-Do not make global find-and-replace changes without first creating an issue and discussing it with us. Global changes can have unintended consequences.
65
+
-Combine multiple small changes (such as minor editorial and technical changes) into a single pull request. This helps us efficiently and effectively facilitate your contribution.
66
+
-Familiarize yourself with the organization and conventions of our existing documentation before creating a pull request. Changes that are consistent with our style and conventions have a higher acceptance rate.
67
67
68
-
- If you need to update the site navigation, ask for help in Slack ([#devdocs](https://magentocommeng.slack.com/messages/CAN932A3H)).
68
+
- If you need to update the site navigation, ask for help in Slack ([#devdocs](https://magentocommeng.slack.com/messages/CAN932A3H)).
69
69
70
-
- Ensure that you update the correct version(s) of documentation (v2.2, v2.3). If you are not sure what directory to put your content in, just do your best. We can help re-locate it (if necessary) during the review process.
71
-
- Review your work for basic typos, formatting errors, or ambiguous sentences before opening a pull request.
72
-
- Revise pull requests according to review feedback. We will close pull requests that require an inordinate amount of time to review and process (especially for minor changes) if you fail to make revisions according to review feedback.
73
-
- Do not directly contact DevDocs team members or maintainers on Slack to review your pull request unless it has been open for more than five days. We have a process and queue for pull requests that everyone must follow.
70
+
-Ensure that you update the correct version(s) of documentation (v2.2, v2.3). If you are not sure what directory to put your content in, just do your best. We can help re-locate it (if necessary) during the review process.
71
+
-Review your work for basic typos, formatting errors, or ambiguous sentences before opening a pull request.
72
+
-Revise pull requests according to review feedback. We will close pull requests that require an inordinate amount of time to review and process (especially for minor changes) if you fail to make revisions according to review feedback.
73
+
-Do not directly contact DevDocs team members or maintainers on Slack to review your pull request unless it has been open for more than five days. We have a process and queue for pull requests that everyone must follow.
74
74
75
75
## Tips for writing content
76
76
77
77
Use the following guidelines to help you with the writing process:
78
78
79
-
- Focus your efforts on providing useful information for your fellow Magento developers and community members. For example, consider providing or revising code samples, important notes, and clarifying vague or ambiguous content.
80
-
- Define the goal of your topic. What exactly do you want to teach the reader?
81
-
- Make the title of your topic reflect the content.
82
-
- Keep your sentences concise. Separate conceptual information from procedural steps.
83
-
- Batch several small changes into a single pull request (instead of separate ones) to ensure your contributions are approved and merged quickly. Have several typo fixes across several areas of documentation? Combine them into a single PR.
84
-
- Remember to write in present tense, use the second person, and use active voice (not passive). For example, _"The log captures commands, output..."_.
85
-
- Use notes to alert readers about important details.
86
-
- Use cross-references to other topics sparingly. We can help you with the syntax if it is not clear. The template provides an example you can use.
87
-
- Need help with markdown? See our [templates](#templates).
79
+
-Focus your efforts on providing useful information for your fellow Magento developers and community members. For example, consider providing or revising code samples, important notes, and clarifying vague or ambiguous content.
80
+
-Define the goal of your topic. What exactly do you want to teach the reader?
81
+
-Make the title of your topic reflect the content.
82
+
-Keep your sentences concise. Separate conceptual information from procedural steps.
83
+
-Batch several small changes into a single pull request (instead of separate ones) to ensure your contributions are approved and merged quickly. Have several typo fixes across several areas of documentation? Combine them into a single PR.
84
+
-Remember to write in present tense, use the second person, and use active voice (not passive). For example, _"The log captures commands, output..."_.
85
+
-Use notes to alert readers about important details.
86
+
-Use cross-references to other topics sparingly. We can help you with the syntax if it is not clear. The template provides an example you can use.
87
+
-Need help with markdown? See our [templates](#templates).
88
88
89
89
### Review changes locally
90
90
@@ -94,11 +94,11 @@ _(Optional)_ To review your changes in HTML output, follow the instructions in t
94
94
95
95
We provide templates to help you get started writing new content and understanding Markdown formatting:
96
96
97
-
-**General topic template** - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/basic_template.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/basic_template.html): This is a template for writing any topic with example formats and styles.
98
-
-**Tutorial templates**: These templates provide example formats and styles for step-by-step instructions (like how-tos). Each file adds navigation buttons when content is generated. Templates include:
99
-
- First introduction topic - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/tutorial-template-first.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/tutorial-template-first.html): Introduction to a tutorial for prerequisites and listing steps
100
-
- Middle topic - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/tutorial-template-middle.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/tutorial-template-middle.html): Use for each step in a tutorial.
101
-
- Final step topic - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/tutorial-template-last.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/tutorial-template-last.html): Use for the last step of the tutorial.
97
+
-**General topic template** - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/basic_template.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/basic_template.html): This is a template for writing any topic with example formats and styles.
98
+
-**Tutorial templates**: These templates provide example formats and styles for step-by-step instructions (like how-tos). Each file adds navigation buttons when content is generated. Templates include:
99
+
- First introduction topic - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/tutorial-template-first.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/tutorial-template-first.html): Introduction to a tutorial for prerequisites and listing steps
100
+
- Middle topic - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/tutorial-template-middle.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/tutorial-template-middle.html): Use for each step in a tutorial.
101
+
- Final step topic - [Markdown](https://github.com/magento/devdocs/blob/master/guides/v2.3/contributor-guide/templates/tutorial-template-last.md) | [HTML](https://devdocs.magento.com/guides/v2.3/contributor-guide/templates/tutorial-template-last.html): Use for the last step of the tutorial.
102
102
103
103
### Edit metadata
104
104
@@ -137,7 +137,7 @@ cd <DEVDOCS_REPOSITORY_ROOT_DIR>
If you have an image that is identical between versions, it is should placed in `/common/images`. Please optimize images before committing them to the repository.
140
+
If you have an image that is identical between versions, it should placed in `/common/images`. Please optimize images before committing them to the repository.
141
141
If done correctly, the symbolic link path will start with 3 or 4 instances of `../`, as the above example shows.
142
142
143
143
### Remove a symbolic link
@@ -168,19 +168,19 @@ You must complete the issue template. We will close your issue if you fail to co
168
168
169
169
You can provide feedback using the following options:
170
170
171
-
- Have general feedback? Create an issue on [GitHub DevDocs](https://github.com/magento/devdocs/issues/new/choose).
172
-
- Have feedback on a specific DevDocs page? Click the **Give us feedback** link at the top right of the page to report on the currently open topic.
171
+
-Have general feedback? Create an issue on [GitHub DevDocs](https://github.com/magento/devdocs/issues/new/choose).
172
+
-Have feedback on a specific DevDocs page? Click the **Give us feedback** link at the top right of the page to report on the currently open topic.
173
173
174
-

174
+

175
175
176
-
- Have a Community code contribution that needs documentation? Create an issue to [request DevDocs content](https://github.com/magento/devdocs/issues/new?template=COMMUNITY_ISSUE_TEMPLATE.md).
176
+
-Have a Community code contribution that needs documentation? Create an issue to [request DevDocs content](https://github.com/magento/devdocs/issues/new?template=COMMUNITY_ISSUE_TEMPLATE.md).
177
177
178
178
## Contact DevDocs
179
179
180
180
Have a question? Need help? Magento DevDocs, Maintainers, and other Contributors are available through:
- Enforce the use of the issue/pull requests template
10
+
- Ask contributors to link to the code base to validate documentation updates when applicable
11
+
- Ask contributors not to contribute to v2.0 docs
12
+
-If a maintainer creates a pull request, it should be reviewed by another maintainer or DevDocs staff member
13
13
14
14
## Labels
15
15
16
16
Labels are important because they help us identify pull requests and ensure that contributors receive points and recognition. See [devdocs awards and points](https://devdocs.magento.com/guides/v2.3/contributor-guide/contributing.html#devdocs-awards-and-points).
17
17
18
18
Here is a brief summary of the most important labels:
19
19
20
-
-`New Topic`: Entirely new documents
21
-
-`Major Update`: Significant new info: new section in existing topic, etc.
22
-
-`Technical`: Changes to technical content/code/processes/naming conventions (any change to technical content)
23
-
-`Editorial`: Typos, grammatical inconsistencies, or minor rewrites
24
-
-`small changes`: See [Small changes workflow](#small-changes-workflow)
20
+
-`New Topic`: Entirely new documents
21
+
-`Major Update`: Significant new info: new section in existing topic, etc.
22
+
-`Technical`: Changes to technical content/code/processes/naming conventions (any change to technical content)
23
+
-`Editorial`: Typos, grammatical inconsistencies, or minor rewrites
24
+
-`small changes`: See [Small changes workflow](#small-changes-workflow)
25
25
26
26
We also use version labels when appropriate.
27
27
@@ -51,8 +51,8 @@ We prefer Markdown over HTML (in most cases). You can use [kramdown](https://kra
51
51
52
52
Before merging a pull request to the `master` branch, it must pass automated testing. Testing takes about 30 minutes to complete for each pull request, so we created a workflow to save time for small changes.
53
53
54
-
-**Individual pull requests to `master`**—15 individual pull requests to `master` x 30 minutes per pull request = 7.5 hrs of testing time
55
-
-**Multiple pull requests using `small_changes`**—1 `small_changes` pull request (containing 15 individual pull requests) to `master` x 30 minutes = 30 minutes of testing time
54
+
-**Individual pull requests to `master`**—15 individual pull requests to `master` x 30 minutes per pull request = 7.5 hrs of testing time
55
+
-**Multiple pull requests using `small_changes`**—1 `small_changes` pull request (containing 15 individual pull requests) to `master` x 30 minutes = 30 minutes of testing time
56
56
57
57
This workflow is for typos, formatting issues, and minor text additions or deletions. It is not for substantial new content, changes to tables, new files, or files that have been moved.
58
58
@@ -63,7 +63,7 @@ Periodically, we will create a pull request from `small_changes` to `master` and
63
63
1. Review the pull request and either approve it or request changes.
64
64
1. Apply the `Small changes` label if one of the following labels should also be applied:
65
65
66
-
-`Editorial`
67
-
-`Technical`
66
+
-`Editorial`
67
+
-`Technical`
68
68
69
-
1. That's it! A devdocs-admin will run tests and merge.
69
+
1. That's it! A devdocs-admin will run tests and merge.
Copy file name to clipboardExpand all lines: .github/PULL_REQUEST_TEMPLATE.md
+2-2Lines changed: 2 additions & 2 deletions
Original file line number
Diff line number
Diff line change
@@ -6,13 +6,13 @@ This pull request (PR) ...
6
6
7
7
<!-- REQUIRED List the affected pages on devdocs.magento.com (URLs). Not needed for large numbers of files. -->
8
8
9
-
- ...
9
+
-...
10
10
11
11
## Links to Magento source code
12
12
13
13
<!-- OPTIONAL - REMOVE THIS SECTION IF NOT USED. If this pull request references a file in a Magento codebase repository, add it here. -->
14
14
15
-
- ...
15
+
-...
16
16
17
17
<!--
18
18
If you are fixing a GitHub issue, note it using GitHub keyword format (https://help.github.com/en/articles/closing-issues-using-keywords#closing-an-issue-in-a-different-repository) to close the issue when this pull request is merged. Example: `Fixes #1234`
0 commit comments