8000 [Intl] [Workflow] fixes English grammar typos by hhamon · Pull Request #34603 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Intl] [Workflow] fixes English grammar typos #34603

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
Nov 26, 2019

Conversation

hhamon
Copy link
Contributor
@hhamon hhamon commented Nov 25, 2019
Q A
Branch? 4.3
Bug fix? yes
New feature? no
Deprecations? no
Tickets ~
License MIT
Doc PR ~

Also related to #34604

@stof
Copy link
Member
stof commented Nov 25, 2019

Please open a PR against the 3.4 branch (oldest supported branch) fixing all occurrences there. Then, we'll need additional PRs for newer branches (4.3 and then 4.4) only for phpdoc which does not exist in older versions (which might mean we don't need the PR at all).

@hhamon
Copy link
Contributor Author
hhamon commented Nov 25, 2019

@stof yes I'll open a PR against 3.4 to fix the other occurrences. This class was introduced in Symfony 4.3 so this is the reason why I isolated the patch in this PR.

@lyrixx
Copy link
Member
lyrixx commented Nov 25, 2019

And that's why I 👍 the PR

@hhamon hhamon force-pushed the symfony-workflow-typo-fix branch from 14ff77a to 492e1b5 Compare November 25, 2019 16:41
@hhamon hhamon changed the title [Workflow] fixes English grammar typo in exception message [Workflow] fixes English grammar typos Nov 25, 2019
@hhamon hhamon changed the title [Workflow] fixes English grammar typos [Intl] [Workflow] fixes English grammar typos Nov 25, 2019
@fabpot
Copy link
Member
fabpot commented Nov 26, 2019

Good catch, thanks @hhamon.

fabpot added a commit that referenced this pull request Nov 26, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

[Intl] [Workflow] fixes English grammar typos

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | ~
| License       | MIT
| Doc PR        | ~

Also related to #34604

Commits
-------

492e1b5 [Intl] [Workflow] fixes English grammar typos
@fabpot fabpot merged commit 492e1b5 into symfony:4.3 Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants
0