Skip to content

Fixed Swiftmailer PHP configuration #11982

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 19, 2019
Merged

Fixed Swiftmailer PHP configuration #11982

merged 1 commit into from
Jul 19, 2019

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