Skip to content

[4.4] Disallow symfony/contracts v2 #34307

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 9, 2019

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Nov 9, 2019

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Travis is red at the moment because unit tests on 4.4 are run against the incompatible event dispatcher contracts v2. https://travis-ci.org/symfony/symfony/jobs/609622341#L4719-L4725

This PR proposes to switch to individual packages, so we can specifically disallow those incompatible contracts.

This PR pins the symfony/contracts package to v1.1 on symfony/symfony.

@derrabus derrabus force-pushed the bugfix/split-contracts branch from 2f29d30 to f2dc2d6 Compare November 9, 2019 19:42
@derrabus derrabus changed the title Require split contracts packages for unit tests [4.4] Disallow symfony/contracts v2 Nov 9, 2019
@derrabus
Copy link
Member Author

derrabus commented Nov 9, 2019

I've reverted to the metapackage. Travis is green, AppVeyor failure looks unrelated.

@nicolas-grekas
Copy link
Member

Thank you @derrabus.

nicolas-grekas added a commit that referenced this pull request Nov 9, 2019
This PR was merged into the 4.4 branch.

Discussion
----------

[4.4] Disallow symfony/contracts v2

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | N/A
| License       | MIT
| Doc PR        | N/A

Travis is red at the moment because unit tests on 4.4 are run against the incompatible event dispatcher contracts v2. https://travis-ci.org/symfony/symfony/jobs/609622341#L4719-L4725

~~This PR proposes to switch to individual packages, so we can specifically disallow those incompatible contracts.~~

This PR pins the `symfony/contracts` package to v1.1 on `symfony/symfony`.

Commits
-------

f2dc2d6 Disallow symfony/contracts v2.
@nicolas-grekas nicolas-grekas merged commit f2dc2d6 into symfony:4.4 Nov 9, 2019
@nicolas-grekas nicolas-grekas added this to the 4.4 milestone Nov 9, 2019
@derrabus derrabus deleted the bugfix/split-contracts branch November 10, 2019 15:15
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