Skip to content

Commit 9b8eb12

Browse files
Copilotdshevtsov
andauthored
Update maintainer's handbook to reflect current contribution policies (#198)
* Initial plan * Update maintainer's handbook with current contribution policies Co-authored-by: dshevtsov <12731225+dshevtsov@users.noreply.github.com> * Peer feedback and editorial changes * Revert the change after SME feedback --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: dshevtsov <12731225+dshevtsov@users.noreply.github.com> Co-authored-by: Dima Shevtsov <shevtsov@adobe.com>
1 parent 5c46cf3 commit 9b8eb12

File tree

1 file changed

+15
-17
lines changed

1 file changed

+15
-17
lines changed

src/pages/guides/maintainers/handbook.md

Lines changed: 15 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -9,43 +9,41 @@ This document describes activities performed by a community maintainer, provides
99

1010
## Repositories and projects
1111

12-
Our maintainers work on numerous repositories within the Magento Github organization. The repository that has most activity is [magento2](https://github.com/magento/magento2). Depending on your interest, you might also interact with some of our other repositories, such as:
12+
Our maintainers work on repositories within the Magento Github organization. Currently, contributions are accepted from the following repositories:
1313

14-
- [pwa-studio](https://developer.adobe.com/commerce/pwa-studio/)
15-
- [magento2-phpstorm-plugin](https://github.com/magento/magento2-phpstorm-plugin)
14+
- [magento2](https://github.com/magento/magento2)
15+
- [magento2-page-builder](https://github.com/magento/magento2-page-builder)
1616
- [inventory](https://github.com/magento/inventory)
17-
- [adobe-stock-integration](https://github.com/magento/adobe-stock-integration)
18-
- and [many others](https://github.com/magento)
1917

20-
Because most contributions go to the [magento2](https://github.com/magento/magento2) repository, there are several GitHub projects to help manage these contributions.
18+
There are several GitHub projects to help manage these contributions.
2119

2220
![Repository and Projects](../../_images/repo-and-projects-v2.png)
2321

24-
Each project has a description of what it does. One important project is the **Community Dashboard**, where you can find the pull requests (PR) that will bring most value for the community and will be delivered first.
22+
Each project has a description of what it does.
2523

2624
## Pull requests dashboard
2725

28-
Within projects there are pull request dashboards. There you will find our Kanban board, which has columns representing the PR delivery workflow. The columns title are self-explanatory and should represent the action required for the pull requests on that column.
26+
Within projects there are pull request dashboards. There you will find our Kanban board, which has columns representing the pull request delivery workflow. The columns title are self-explanatory and should represent the action required for the pull requests on that column.
2927

3028
![pull request Dashboard](../../_images/pr-dashboard.png)
3129

3230
## Pull request review process
3331

34-
The first step in the review process is to choose and self-assign a pull request from the dashboard. To help maintainers choose a PR to review, there are several labels, the most important being **Priority**. We encourage our maintainers to choose pull requests with the highest priority from the **Community Dashboard** in the **Pending Review** column.
32+
The first step in the review process is to choose and self-assign a pull request from the dashboard. To help maintainers choose a pull request to review, there are several labels, the most important being **Priority**. We encourage our maintainers to choose pull requests with the highest priority from the **Pending Review** column.
3533

3634
Once you have chosen a pull request, there are some steps to be followed during the review process.
3735

38-
### Check CLA and builds
36+
### Check Contributor License Agreement and builds
3937

40-
The first thing we encourage a maintainer to check is if the contributor has signed the Adobe CLA. Without this signature, we cannot accept the contribution. If the Contributor has not signed the CLA, that check will be red. Add a comment to the pull request requesting the contributor to sign the CLA.
38+
The first thing we encourage a maintainer to check is if the contributor has signed the Adobe Contributor License Agreement (CLA). Without this signature, we cannot accept the contribution. If the Contributor has not signed the CLA, that check will be red. Add a comment to the pull request requesting the contributor to sign the CLA.
4139

42-
Once the CLA has been signed, we can check the other builds. Those builds are run based on the contributors pull request and will let us know if the proposed changes are causing existing functionality to break or are not fully compliant with coding standards. If either the Heath Index or Semantic Version Checker fail, the PR should be changed or the proposed changes will need approval by the Adobe team. More information can be found on our [Contributor Guide](../code-contributions/pull-request-tests.md).
40+
Once the CLA has been signed, we can check the other builds. Those builds are run based on the contributors pull request and will let us know if the proposed changes are causing existing functionality to break or are not fully compliant with coding standards. If either the Heath Index or Semantic Version Checker fail, the pull request should be changed or the proposed changes will need approval by the Adobe team. More information can be found on our [Contributor Guide](../code-contributions/pull-request-tests.md).
4341

44-
![Check CLA and Builds](../../_images/builds-and-checks.png)
42+
![Check Contributor License Agreement and Builds](../../_images/builds-and-checks.png)
4543

4644
### Check the pull request target
4745

48-
It is important to ensure that pull requests are targeted to the correct branch. Currently, on the Magento2 repository, we are processing only those PRs that are aimed at the 2.4-develop branch. Please make sure your PR targets this branch.
46+
It is important to ensure that pull requests are targeted to the correct branch. Currently, on the Magento2 repository, we are processing only those pull requests that are aimed at the 2.4-develop branch. Please make sure your pull request targets this branch.
4947

5048
![Check pull request Target](../../_images/pr-target-branch.png)
5149

@@ -69,7 +67,7 @@ If the pull request is not covered by tests, the maintainer should advise the co
6967

7068
### Approve changes
7169

72-
Once all the steps above are complete, the maintainer can approve the contributor’s PR. After approving the PR, it will proceed through the delivery process and will be tested to guarantee quality.
70+
Once all the steps above are complete, the maintainer can approve the contributor's pull request. After approving the pull request, it will proceed through the delivery process and will be tested to guarantee quality.
7371

7472
![Approve Changes](../../_images/approve-changes.png)
7573

@@ -79,9 +77,9 @@ Besides the repositories, projects and code review process, there are other tool
7977

8078
### Related pull requests
8179

82-
Magento Open source is a complex platform and, some changes may require changes in multiple repositories. For example, if a contributor’s PR performs a change on a feature that is being used on the Adobe Commerce edition, it may require a parallel PR in that repository.
80+
Magento Open source is a complex platform and, some changes may require changes in multiple repositories. For example, if a contributor's pull request performs a change on a feature that is being used on the Adobe Commerce edition, it may require a parallel pull request in that repository.
8381

84-
In that case, builds need to run using the changes from both PRs. To do so, use the 'related pull requests' feature. This feature is enabled by adding the link to the related pull request on the main pull request description using Github keywords. Details on this are in the [Contributor Guide](../code-contributions/pull-request-tests.md#related-pull-requests).
82+
In that case, builds need to run using the changes from both pull requests. To do so, use the 'related pull requests' feature. This feature is enabled by adding the link to the related pull request on the main pull request description using Github keywords. Details on this are in the [Contributor Guide](../code-contributions/pull-request-tests.md#related-pull-requests).
8583

8684
![Related pull requests](../../_images/related-prs.png)
8785

0 commit comments

Comments
 (0)