-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Use valid version in maintainer guide #13175
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
√
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this attention to detail ... but this document is for internal use only, so it's not critical to keep it updated like this. In any case, thanks for this contribution Oskar.
@@ -195,7 +195,7 @@ Step 3: Merge it into the other branches | ||
|
||
If a PR has not been merged in ``master``, you must merge it up into all the | ||
maintained branches until ``master``. Imagine that you are merging a PR against | ||
``3.4`` and the maintained branches are ``3.4``, ``4.2`` and ``master``: | ||
``3.4`` and the maintained branches are ``3.4``, ``4.4`` and ``master``: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add 5.0 though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do this while upmerging 👍🏻
This PR was merged into the 3.4 branch. Discussion ---------- Use valid version in maintainer guide <!-- If your pull request fixes a BUG, use the oldest maintained branch that contains the bug (see https://symfony.com/roadmap for the list of maintained branches). If your pull request documents a NEW FEATURE, use the same Symfony branch where the feature was introduced (and `master` for features of unreleased versions). --> Commits ------- 2a596de Use valid version in maintainer guide
* 4.4: Update http_authentication.rst Update routing.rst Update routing.rst Update mailer.rst Update messenger.rst Reference method Update form.rst Use valid versionj in maintainer guide. refs #13175 [DependencyInjection] Fixed public service use case fixup tweak #13099 Use valid version in maintainer guide Mention region parameter for Mailgun mailer too Update service_subscribers_locators.rst
* 5.0: Update http_authentication.rst Update routing.rst Update routing.rst Update mailer.rst Update messenger.rst Reference method Update form.rst Use valid versionj in maintainer guide. refs #13175 [DependencyInjection] Fixed public service use case Use valid version in maintainer guide Mention region parameter for Mailgun mailer too
No description provided.