Skip to content
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

Update doctrine dependencies to newest version #1149

Merged
merged 1 commit into from Mar 4, 2014
Merged

Update doctrine dependencies to newest version #1149

merged 1 commit into from Mar 4, 2014

Conversation

QuingKhaos
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #1143, #1148
License MIT
Doc PR n/a

Update doctrine dependencies to newest version and add cache-bundle at dev. Since doctrine/DoctrineBundle@e37301f7 it depends on the new doctrine/doctrine-cache-bundle: ~1.0 which is currently not available as stable version.

Without this change composer update would degrade to doctrine/doctrine-bundle: 1.3.0-beta1 which is known as broken.

References doctrine/DoctrineBundle#276 and doctrine/DoctrineCacheBundle#13 (after releasing this fix can be removed ^^)

@QuingKhaos
Copy link
Contributor Author

~~Is the doctrine-bundle: ~1.3@dev still needed? ~~It seems all occurrences of theDoctrineOrmMappingPass are gone, maybe a downgrade to ~1.2 is possible?~~ [Uff, typo while searching the code, it's named DoctrineOrmMappingsPass (beware of the plural ^^).]

@Sylius, exposed from #1141, #931 becomes actual, only the bundles which uses the mappings pass should declare the ~1.3 dependency and not the resource-bundle as in #870 changed. This prevents using the resource-bundle standalone and with a stable doctrine-bundle version.

@QuingKhaos
Copy link
Contributor Author

ping @pjedrzejewski what do you think?

pjedrzejewski pushed a commit that referenced this pull request Mar 4, 2014
Update doctrine dependencies to newest version
@pjedrzejewski pjedrzejewski merged commit 2bd66e9 into Sylius:master Mar 4, 2014
@pjedrzejewski
Copy link
Member

Sounds right, thank you Patrik! I'll have a look at the other issues.

@QuingKhaos QuingKhaos deleted the fix/doctrine-deps branch May 14, 2014 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants