Skip to content

Conversation

@xabbuh
Copy link
Member

@xabbuh xabbuh commented Jun 23, 2016

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

array('"test".test@iana.org'),
array('"test"' . chr(0) . '@iana.org'),
array('"test\"@iana.org'),
array(chr(226) . '@iana.org'),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spaces around . should be removed

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fabbot is happy now.

@xabbuh xabbuh force-pushed the email-validator-2.0-support branch from 783cbc7 to 7a9c231 Compare June 23, 2016 12:44
@xabbuh xabbuh force-pushed the email-validator-2.0-support branch from 7a9c231 to 25d39aa Compare June 23, 2016 12:45
@fabpot
Copy link
Member

fabpot commented Jun 23, 2016

Thank you @xabbuh.

@fabpot fabpot merged commit 25d39aa into symfony:master Jun 23, 2016
fabpot added a commit that referenced this pull request Jun 23, 2016
This PR was merged into the 3.2-dev branch.

Discussion
----------

[Validator] support egulias/email-validator 2.x

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

Commits
-------

25d39aa support egulias/email-validator 2.x
@xabbuh xabbuh deleted the email-validator-2.0-support branch June 23, 2016 13:38
@fabpot fabpot mentioned this pull request Oct 27, 2016
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.

3 participants