8000 [Validator] Backported constraint validator tests from 2.5 by webmozart · Pull Request #11646 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Validator] Backported constraint validator tests from 2.5 #11646

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
Aug 14, 2014

Conversation

webmozart
Copy link
Contributor
Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

This PR backports the constraint validator tests from 2.5 to 2.3 to facilitate maintenance. When we adapt a test in 2.3, we can now easily merge the change forward to 2.5.

@webmozart
Copy link
Contributor Author

ping @stof @Tobion

@stof
Copy link
Member
stof commented Aug 12, 2014

👍

1 similar comment
@nicolas-grekas
Copy link
Member

👍

@nicolas-grekas
Copy link
Member

@webmozart can you rebase please?

@nicolas-grekas
Copy link
Member

After the rebase, are you ok @symfony/deciders to let me merge this PR, then merge 2.3 into 2.4 to allow @webmozart to finish his #11647 PR?

@stof
Copy link
Member
stof commented Aug 13, 2014

@nicolas-grekas yeah, IMO it is fine (anyway, once it is rebased, @webmozart is allowed to merge the PR as he is merger for the component and 2 other deciders have validated it)

@romainneutron
Copy link
Contributor

👍

@webmozart webmozart merged commit 87a47ea into symfony:2.3 Aug 14, 2014
webmozart added a commit that referenced this pull request Aug 14, 2014
….5 (webmozart)

This PR was merged into the 2.3 branch.

Discussion
----------

[Validator] Backported constraint validator tests from 2.5

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

This PR backports the constraint validator tests from 2.5 to 2.3 to facilitate maintenance. When we adapt a test in 2.3, we can now easily merge the change forward to 2.5.

Commits
-------

87a47ea [Validator] Backported constraint validator tests from 2.5
@webmozart webmozart deleted the backport-tests-to-2.3 branch August 15, 2014 09:42
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.

4 participants
0