Proposing a Change

Install the Software Stack

Before working on Symfony, setup a friendly environment with the following software:

• Git;
• PHP version 5.5.9 or above.

Configure Git

Set up your user information with your real name and a working email address:

1
2

$ git config --global user.name "Your Name"
$ git config --global user.email you@example.com

1

$ git config core.autocrlf

1

$ git config --global core.autocrlf input

Get the Symfony Source Code

Get the Symfony source code:

• Create a GitHub account and sign in;
• Fork the Symfony repository (click on the "Fork" button);
• After the "forking action" has completed, clone your fork locally (this will create a symfony directory):

1

$ git clone git@github.com:USERNAME/symfony.git

• Add the upstream repository as a remote:

1
2

$ cd symfony
$ git remote add upstream git://github.com/symfony/symfony.git

Check that the current Tests Pass

Now that Symfony is installed, check that all unit tests pass for your environment as explained in the dedicated document.

The License

Before you start, you should be aware that all the code you are going to submit must be released under the MIT license.

Choose the right Branch

Before working on a PR, you must determine on which branch you need to work:

• 3.4, if you are fixing a bug for an existing feature or want to make a change that falls into the list of acceptable changes in patch versions (you may have to choose a higher branch if the feature you are fixing was introduced in a later version);

• 5.x, if you are adding a new feature.

  The only exception is when a new major Symfony version (4.0, 5.0, etc.) comes out every two years. Because of the special development process of those versions, you need to use the previous minor version for the features (e.g. use 3.4 instead of 4.0, use 4.4 instead of 5.0, etc.)

Create a Topic Branch

Each time you want to work on a PR for a bug or on an enhancement, create a topic branch:

1

$ git checkout -b BRANCH_NAME 5.x

Or, if you want to provide a bug fix for the 3.4 branch, first track the remote 3.4 branch locally:

1

$ git checkout --track origin/3.4

Then create a new branch off the 3.4 branch to work on the bug fix:

1

$ git checkout -b BRANCH_NAME 3.4

The above checkout commands automatically switch the code to the newly created branch (check the branch you are working on with git branch).

Use your Branch in an Existing Project

If you want to test your code in an existing project that uses symfony/symfony or Symfony components, you can use the link utility provided in the Git repository you cloned previously. This tool scans the vendor/ directory of your project, finds Symfony packages it uses, and replaces them by symbolic links to the ones in the Git repository.

1

$ php link /path/to/your/project

Before running the link command, be sure that the dependencies of the project you want to debug are installed by running composer install inside it.

Work on your Pull Request

Work on the code as much as you want and commit as much as you want; but keep in mind the following:

• Read about the Symfony conventions and follow the coding standards (use git diff --check to check for trailing spaces -- also read the tip below);
• Add unit tests to prove that the bug is fixed or that the new feature actually works;
• Try hard to not break backward compatibility (if you must do so, try to provide a compatibility layer to support the old way) -- PRs that break backward compatibility have less chance to be merged;
• Do atomic and logically separate commits (use the power of git rebase to have a clean and logical history);
• Never fix coding standards in some existing code as it makes the code review more difficult;
• Write good commit messages (see the tip below).

Prepare your Pull Request for Submission

When your PR is not about a bug fix (when you add a new feature or change an existing one for instance), it must also include the following:

• An explanation of the changes in the relevant CHANGELOG file(s) (the [BC BREAK] or the [DEPRECATION] prefix must be used when relevant);
• An explanation on how to upgrade an existing application in the relevant UPGRADE file(s) if the changes break backward compatibility or if you deprecate something that will ultimately break backward compatibility.

Rebase your Pull Request

Before submitting your PR, update your branch (needed if it takes you a while to finish your changes):

1
2
3
4
5

$ git checkout 5.x
$ git fetch upstream
$ git merge upstream/5.x
$ git checkout BRANCH_NAME
$ git rebase 5.x

When doing the rebase command, you might have to fix merge conflicts. git status will show you the unmerged files. Resolve all the conflicts, then continue the rebase:

1
2

$ git add ... # add resolved files
$ git rebase --continue

Check that all tests still pass and push your branch remotely:

1

$ git push --force origin BRANCH_NAME

Make a Pull Request

You can now make a pull request on the symfony/symfony GitHub repository.

To ease the core team work, always include the modified components in your pull request message, like in:

1
2

[Yaml] fixed something
[Form] [Validator] [FrameworkBundle] added something

The default pull request description contains a table which you must fill in with the appropriate answers. This ensures that contributions may be reviewed without needless feedback loops and that your contributions can be included into Symfony as quickly as possible.

Some answers to the questions trigger some more requirements:

• If you answer yes to "Bug fix?", check if the bug is already listed in the Symfony issues and reference it/them in "Fixed tickets";
• If you answer yes to "New feature?", you must submit a pull request to the documentation and reference it under the "Doc PR" section;
• If you answer yes to "BC breaks?", the PR must contain updates to the relevant CHANGELOG and UPGRADE files;
• If you answer yes to "Deprecations?", the PR must contain updates to the relevant CHANGELOG and UPGRADE files;
• If you answer no to "Tests pass", you must add an item to a todo-list with the actions that must be done to fix the tests;
• If the "license" is not MIT, just don't submit the pull request as it won't be accepted anyway.

If some of the previous requirements are not met, create a todo-list and add relevant items:

1
2
3

- [ ] fix the tests as they have not been updated yet
- [ ] submit changes to the documentation
- [ ] document the BC breaks

If the code is not finished yet because you don't have time to finish it or because you want early feedback on your work, add an item to todo-list:

1
2

- [ ] finish the code
- [ ] gather feedback for my changes

As long as you have items in the todo-list, please prefix the pull request title with "[WIP]". If you do not yet want to trigger the automated tests, you can also set the PR to draft status.

In the pull request description, give as much detail as possible about your changes (don't hesitate to give code examples to illustrate your points). If your pull request is about adding a new feature or modifying an existing one, explain the rationale for the changes. The pull request description helps the code review and it serves as a reference when the code is merged (the pull request description and all its associated comments are part of the merge commit message).

In addition to this "code" pull request, you must also send a pull request to the documentation repository to update the documentation when appropriate.

Rework your Pull Request

Based on the feedback on the pull request, you might need to rework your PR. Before re-submitting the PR, rebase with upstream/5.x or upstream/3.4, don't merge; and force the push to the origin:

1
2

$ git rebase -f upstream/5.x
$ git push --force origin BRANCH_NAME

Moderators earlier asked you to "squash" your commits. This means you will convert many commits to one commit. This is no longer necessary today, because Symfony project uses a proprietary tool which automatically squashes all commits before merging.