Skip to content

Conversation

@dborsatto
Copy link
Contributor

Just a fix for a PHP snippet in the Swiftmailer reference configuration. I chose 4.0 as target as it's the branch when this snippet was introduced, let me know if I need to change anything.

@gregurco
Copy link
Contributor

Hello.

I guess this PR should be for version 4.2 following this rule:

Fixes should always be based on the oldest maintained branch which contains the error.

4.0 is not maintained version.

Ref: https://symfony.com/doc/current/contributing/documentation/overview.html

@javiereguiluz javiereguiluz added this to the 4.2 milestone Jul 19, 2019
@javiereguiluz javiereguiluz changed the base branch from 4.0 to 4.2 July 19, 2019 14:07
@javiereguiluz
Copy link
Member

Thanks Davide! @gregurco is right and this must go to 4.2 branch, so we changed the branch while merging. Congrats on your first Symfony Docs contribution!

@javiereguiluz javiereguiluz merged commit e1ad33e into symfony:4.2 Jul 19, 2019
javiereguiluz added a commit that referenced this pull request Jul 19, 2019
This PR was submitted for the 4.0 branch but it was merged into the 4.2 branch instead (closes #11982).

Discussion
----------

Fixed Swiftmailer PHP configuration

Just a fix for a PHP snippet in the Swiftmailer reference configuration. I chose 4.0 as target as it's the branch when this snippet was introduced, let me know if I need to change anything.

Commits
-------

e1ad33e Fixed Swiftmailer PHP configuration
@dborsatto
Copy link
Contributor Author

Hey, sorry for not following up yesterday, I was busy and you beat me to it. Apologies for initially targeting the wrong branch, but I'm happy to have made a (small) contribution ☺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants