Code Contributions

The following topics are included in this guide:

• Contribute to Magento 2 code
• GitHub and Two-Factor authentication
• Questions or enhancement requests?
• Accepted pull requests and ported code
• Contribution requirements
• Forks and pull requests
• Squash commits
• Magento Contributor Assistant
• Report an issue
• Help triage issues
• Labels applied by the Community Engineering team
• Contribution awards and points

Connect with Magento Contributors and Maintainers to learn more about Magento contributions and special projects. Join us in our Slack workspace, in the #general channel, follow the #announcements, and browse for more channels!

Contribute to Magento 2 code

We use the fork and pull model to contribute to the Magento 2 codebase. This method allows contributors to maintain their own copy of the forked codebase, which can be easily synced with the main copy. The forked repository is then used to submit a pull request to the base repository to merge a set of changes from the fork into the main repository.

Contributions can take the form of new components or features, changes to existing features, tests, documentation (such as developer guides, user guides, examples, or specifications), bug fixes, optimizations, or just good suggestions.

The Community Engineering Team reviews all issues and contributions submitted by the community developers. During the review we might require clarifications from the contributor. If there is no response from the contributor in two weeks (14 days) time, the issue is closed.

When the Community Engineering Team works on reviewing the suggested changes, we will add a label to the issue to indicate certain information, like the status or who is working the issue. See Labels to learn more.

Refer to Magento Contributor Agreement for detailed information about the License Agreement. All contributors are required to submit a click-through form to agree to the terms.

GitHub and Two-Factor authentication

Magento requires all Partners who contribute code to enable 2FA on their GitHub accounts. You can use a mobile device or 2FA application for added protection. See Configuring two-factor authentication in the GitHub help.

We also recommend creating a personal access token for your account to use when interacting with GitHub in scripts and on the command line. See Creating a personal access token for the command line in the GitHub help.

Questions or enhancement requests?

We capture code-related issues in the the Magento 2 repo and documentation-related issues in the Magento DevDocs repo. If you have questions about Magento functionality or processes, we recommend posting them to a question-and-answer site, such as Stack Exchange and the Magento Forums, where Magento community members can quickly provide recommendations and advice.

Submit feature requests or enhancement suggestions to the Magento 2 Feature Requests and Improvements forum. For details about how requests are managed, see Improvements to GitHub Management.

Accepted pull requests and ported code

v2.4 Contribution Notice: Magento is focusing development efforts on v2.4, so we are no longer accepting pull requests to the v2.3, v2.2, v2.1, and v2.0 release lines.

Review the following supported and accepted pull request rules. We defined these rules to simplify and accelerate your submissions, ensure code consistency, manage current and backlog tasks, and so on.

	Fix for Existing Issue	Test Coverage	Refactoring	New Feature	Code Cleanup
2.1
2.2
2.3
2.4

Contribution requirements

1. Contributions must adhere to Magento coding standards.
2. Refer to the Magento development team’s Definition of Done. We use these guidelines internally to ensure that we deliver well-tested, well-documented, and solid code. We encourage you to use this as well!
3. Pull requests (PRs) must be accompanied by a meaningful description of their purpose. Comprehensive descriptions increase the chances that a pull request is merged quickly and without additional clarification requests.
4. Commits must be accompanied by meaningful commit messages.
5. PRs that include bug fixes must be accompanied by a step-by-step description of how to reproduce the bug.
6. PRs that include new logic or new features must be submitted along with:
   • Unit/integration test coverage (we will be releasing more information about writing test coverage in the near future).
   • Follow the Magento DevDocs contribution work flow and guidelines to submit proposed documentation updates to the Magento DevDocs Github repository.
7. For large features or changes, open an issue to discuss your proposal first. Notifying us in advance can prevent duplicate or unnecessary effort, and also offers an opportunity to get additional background information and help from other contributors.
8. To report a bug, open an issue and follow the Issue reporting guidelines.
9. Verify that all automated tests on your pull request pass successfully.

Forks and pull requests

For complete information about contributing to Magento projects, see the Beginner Guides on the Magento 2 repository. These guides help you:

• Fork a repository
• Create a branch
• Find and work issues
• Create tests
• Submit a pull request

Squash commits

Sometimes your pull request may have more than one commit (the main commit, then changes to it after review, etc). A good practice is to deliver commits that bring finalized, functional parts/bugfixes. In that case, all intermediate commits like “static test fix”, “typo fix”, “minor refactoring” should be squashed into a single commit. This helps keep a clean history and makes the repo easier to read. There is no requirement to have only one commit per PR. However, the intermediate commits in most cases bring no value into the commits history, which is why it is a good to keep the history clean and useful.

Magento contributor assistant

The Magento Contributor Assistant is a bot that currently runs on the GitHub magento/magento2 repository. It helps automate different issue and pull request workflows using commands entered as comments.

Currently, the Magento Contributor Assistant automatically deploys test instances on Magento’s hosting based on a contributor’s pull request or provide a vanilla Magento instance. This gives a GitHub user an instance to test pull requests or reported issues. We plan on adding features in the future.

• Deploy vanilla Magento instance
• Deploy instance based on PR changes
• Combine multiple pull requests

Deploy vanilla Magento instance

When you need to verify an issue or pull request, enter a command to generate an issue verification instance, or vanilla Magento. This instance is a clean Magento installation of a specified version tag or the develop branch of a specified release line.

Command: To deploy a vanilla Magento instance, add the following command as a comment to the GitHub Pull Request or Issue:

1
@magento give me {$version} instance

For version, the currently supported values are latest version tags and 2.4-develop branch.

Actions: The following actions complete for the command:

• If the instance does not exist, it will be deployed. Deployment takes ~2 minutes.
• If the instance exists, a fresh instance will be redeployed.
• By default, instances have a lifetime of 3 hours. All deployments are terminated after that.

Admin access:

• https://i-{$issue_number}.{branch}.instances.magento-community.engineering/admin
• Admin Credentials:
  • Username: admin
  • Password: 123123q

Permissions:

• All permissions granted for all users.

Deploy instance based on PR changes

To verify and test changes completed in a pull request, enter a command to generate a Magento instance using the code based in the PR.

Command: To deploy, Community Maintainers, a Magento EngCom Team member, or contributor under the existing Pull Request enters the following command as a comment to the pull request:

1
@magento give me test instance

Actions:

• It deploys a new Magento instance based on Pull Request changes.
• Deployment takes ~2 minutes.

Admin access:

• http://pr-xxx.engcom.dev.magento.com/admin
• Admin Credentials:
  • Username: admin
  • Password: 123123q

Permissions:

• Community Maintainers
• Magento EngCom Team
• Contributor

Combine multiple pull requests

To optimize the pull request queue, enter a command with a series of related pull requests to verify and combine the code. If all tests pass, the entered PRs are merged into the current PR.

Command: To combine pull requests, a member of the Community Maintainers or Magento EngCom Team under the existing Pull Request enters the following command:

1
@magento combine {xxx} {yyy} {zzz}

The command merges the listed related pull requests (xxx, yyy, zzz) into the current pull request. For example: @magento combine 1234 1238 1239

actions: When all conditions are passed, all related pull requests will be closed and merged to the current PR:

• Current pull request allows changes from maintainers.
• All mentioned pull requests are open.
• All mentioned pull requests have been created by the same contributor (author).
• All mentioned pull requests have same target (base) branch.
• All mentioned pull requests can be merged with each other.

Permissions:

• Community Maintainers
• Magento EngCom Team

Report an issue

If you find a bug in Magento 2 code, you can report it by creating an issue in the Magento 2 repository.

Before creating an issue:

1. Read the Issue reporting guidelines to learn how to create an issue that can be processed in a timely manner.
2. Check the documentation to make sure the behavior you are reporting is really a bug, not a feature.
3. Review the existing issues to make sure you are not duplicating another contributor’s work.

To add an issue:

1. In the Magento 2 public repository, click the Issues tab.

   

2. Click New issue.

   

3. Select a type of issue: Bug report, Developer experience issue, or Feature request.
4. Fill in the Title, description, and additional information for the template.
5. Click Submit new issue.

When you submit the issue, a validation process begins. If the issue doesn’t have enough information, you as the Reporter may need to add more information. See GitHub Issues Processing Workflow for complete details on issue verification.

Help triage issues

In addition to contributing code, you can help triage issues. This can include reproducing bug reports or asking for vital information, such as affected versions or instructions to reproduce bugs. If you want to triage issues, you can begin by subscribing to Magento on CodeTriage.

Labels applied by the Community Engineering team

We apply labels to public pull requests and issues to help other participants retrieve additional information about current progress, component assignments, Magento release lines, and much more. The following information details global labels used in Magento 2 repositories and across Community Engineering contributions.

Release Lines

Release line labels indicate the specific Magento release lines affected by the issue or PR. For example, if working on a fix for 2.4.0 you would apply the Release Line: 2.4. This effectively includes all releases in this line.

• Release Line: 2.3
• Release Line: 2.4

Progress

Progress labels indicate the Pull Request status on each review stage:

• Progress: needs update - The Community Engineering Team needs additional information from the reporter to properly prioritize and process the pull request.
• Progress: on hold - The pull request is on hold due and will be further reviewed to accept or reject.
• Progress: accept - The pull request has been accepted and will be merged into mainline code.
• Progress: reject - The pull request has been rejected and will not be merged into mainline code. Possible reasons can include but are not limited to: issue has already been fixed in another code contribution, or there is an issue with the code contribution.

Contribution awards

The level of investigation, research, and work required for a task may differ. Contribution Rewards labels indicate what type of contribution awards will be applied when completing an issue and PR. Some awards will provide higher points and rewards than others.

• Award: complex
• Award: advanced
• Award: special achievement
• Award: category of expertise
• Award: test coverage
• Award: devdocs update
• Award: MFTF test coverage
• Award: bug fix
• Cleanup
• Port

Partners

All partners Pull Requests should be marked with label partners-contribution. Additionally, add a partner label for PRs submitted by specific Partners. Use the format: Partner: <PartnerName>. The following are Partner examples:

Example labels:

• partners-contribution
• Partner: Atwix
• Partner: Comwrap
• Partner: Interactiv4
• Partner: Wagento

Components

Component labels indicate the components affected by the Pull Request. To learn more about available components and assigned architects, see Magento Components Assignment.

Example labels:

• Component: Catalog
• Component: Report
• Component: Checkout

For edge cases, Component: Other and Component: Multiple may be used.

Events

Event labels mark recommended issues and submitted PRs for a specific event. Events may include Contribution Days, Hackathons, Imagine, special events like Smashtoberfest, and others. Contributors and Maintainers can easily locate code when attending those events. Some events may also have a Community Engineering Slack channel using the same label.

Example labels:

• Event: mm18in
• Event: mm17es
• Event: mlau18

General

General labels include a variety of tasks and definitions for pull requests and issues.

• good first issue - Indicates a good issue for first-time contributors.
• help wanted - Indicates the creator or author needs help with a decision, advice for resolving, and so on.
• triage wanted - Indicates the issues are under triage. See this information to learn more about the Triage Wanted program.

Issue resolution status

Labels applied to issues through verification and completion. For details on the process, see GitHub Issues Processing Workflow.

• Issue: Format is not valid - Gate 1 failed. Automatic verification by the Automated Contributor Assistant failed and the issue needs updates. The format of the issue description and minimum required information is not provided: Preconditions, Steps to Reproduce, Actual Result, Expected Result. Previous label G1 Failed.
• Issue: Format is valid - Gate 1 passed. Automatic verification by the Automated Contributor Assistant passed for all issue content. Previous label G1 Passed.
• Issue: Clear Description - Gate 2 passed. The Community Engineering Team has confirmed that this issue contains the minimum required information to reproduce. Previous label G2 Passed.
• Issue: Cannot Reproduce - Gate 3 failed. The issue could not be reproduced or validated. Previous label Cannot Reproduce.
• Issue: Confirmed - Gate 3 passed. Manual verification of the issue description and reproduction steps was confirmed. Previous label G3 Passed.
• Issue: Ready for Work - Gate 4 passed. The issue is acknowledged and added to the backlog for open development. Previous label acknowledged.
• Reproduced on 2.3.x - The Community Engineering Team reproduced the issue on latest 2.3.x release.

• Reproduced on 2.4.x - The Community Engineering Team reproduced the issue on latest 2.4.x release.

• Fixed in 2.3.x - The issue has been fixed in one of the 2.3.x releases.
• Fixed in 2.4.x - The issue has been fixed in one of the 2.4.x releases or in 2.4-develop branch and will be available with the upcoming patch release.
• non-issue - A described behavior in the issue description is valid and shouldn’t be changed in Magento code base.

DevDocs

All contributions to DevDocs receive the following labels:

• New topic- New topic submissions for content that has never existed on DevDocs such as tutorials, references, instructions, and so on
• Major update - Significant original updates to existing content
• Technical - Updates to the code or processes that alter the technical content of the document, such as code snippets, reference documentation, parameter names and values, and other relevant content
• Editorial - Fixes for typos, grammatical inconsistencies, or minor rewrites to correct inaccuracies

Contribution awards and points

We are thankful for all contributions, and we always recognize our most active members. Our aim is to find and recognize our top contributors according to points awarded during a given time period (monthly/quarterly/yearly). Contributors can earn points in numerous ways with a focus on pull requests (PR) to the backlog and special projects.

The Community Engineering team assesses each PR and determines the best awards for the submitted work. Contribution points are calculated according to the assessment results. We award points when the PR is merged.

How points are awarded

Every merged PR receives one base achievement and potentially one, or more, additional achievements. These achievements are applied to PRs during review and assessment. Contributors and maintainers receive points after the PR is merged.

Due to the level of work required for developing and reviewing a PR, contributors and maintainers receive the same amount of points.

For instance, see this calculated reward points example:

1. When a contributor submits a PR with complex code contributions, they receive a total of 30 points:
   • Improvement (base achievement) = 10 points
   • Complex (additional achievement) = 20 points
2. When a contributor submits a PR with complex code and tests, they receive a total of 40 points:
   • Improvement (base achievement) = 10 points
   • Complex (additional achievement) = 20 points
   • Test coverage (additional achievement) = 10 points
3. When a contributor submits a PR port of an existing merged PR, they receive a total of 35 points:
   • Improvement (base achievement) = 10 points
   • Complex (additional achievement) = 20 points
   • Author of ported issue = 5 points
4. When a maintainer reviews and approves a PR with complex code and tests, they receive a total of 40 points:
   • Improvement (base achievement) = 10 points
   • Complex (additional achievement) = 20 points
   • Test coverage (additional achievement) = 10 points

Earned achievements display as labels in each PR in GitHub and for each contributor, partner, and maintainer on magento.com.

Multiple minor editorial fixes should be combined into a single PR, for the sake of efficiency.

Base achievements

Every merged PR receives one base achievement.

Additional achievements

Every PR may receive several additional achievements during assessment.

DevDocs awards and points

Contributors and maintainers can also earn rewards for merged submissions to the DevDocs repository. These earned points add to contributor totals. Like code contributions, the DevDocs team assesses the PRs and applies labels to determine the level of work and achievements. Each PR receives one base achievement and potentially additional achievements.

These examples show how reward points are calculated:

• A contributor who submits a PR that reformats HTML into markdown or fixes typos will receive an Editorial (base achievement) label = 1 point
• A contributor who submits a PR with a code sample update and new parameter descriptions will receive an Improvement (base achievement) label = 10 points and either a Major update (additional achievement) label = 20 points OR a Technical label (additional achievement) = 10 points

Base achievements

Every merged PR to the DevDocs repository receives one base achievement. If you enter a PR with editorial fixes and new content, you receive the Improvement and additional achievements.

Additional achievements

Each PR can earn one of the following additional achievements. If you entered an editorial PR, you may not receive an additional achievement.

If the PR earns recognition for significant updates and additions to the documentation, it may also receive a Special Achievement label, which garners an additional 20 points. This is determinable at the time of internal assessment. Certain issues may already be labeled as Special Achievement in an effort to prompt contribution—the PRs that fix these issues will receive the extra 20 points.