Skip to content
This repository was archived by the owner on Mar 19, 2024. It is now read-only.

Symfony 4 support #13

Merged
merged 2 commits into from
Sep 17, 2018
Merged

Conversation

chirimoya
Copy link
Collaborator

@chirimoya chirimoya commented May 31, 2018

@dantleech
Copy link
Owner

Hmm looks like the Symfony CMF testing package needs SF4 support too.

Not sure about the memory issue on 5.x, but happy to drop 5.x from Travis.

Then it seems that we use a deprecated Doctrine ORM method - that would need to be looked at (but may well be in the CMF testing).

Also happy to drop the CMF testing dep if possible.

@chirimoya
Copy link
Collaborator Author

@dantleech symfony-cmf/testing 2.1.0-RC1 should work with symfony 4.0. will work on this PR after phpcr/phpcr-migrations#8 is merged.

@chirimoya chirimoya force-pushed the symfony-4-support branch 4 times, most recently from 801f313 to 9db3197 Compare June 4, 2018 13:28
@chirimoya chirimoya force-pushed the symfony-4-support branch 9 times, most recently from dfb8e7b to 1b11d6f Compare June 4, 2018 16:18
@chirimoya chirimoya changed the title [WIP] Symfony 4 support Symfony 4 support Sep 17, 2018
@chirimoya
Copy link
Collaborator Author

@dantleech can you have a look? tests are green now.

@chirimoya chirimoya requested a review from dantleech September 17, 2018 10:13
@dantleech dantleech merged commit 667b7f4 into dantleech:master Sep 17, 2018
@dantleech
Copy link
Owner

Do you guys want to maintain?

@chirimoya chirimoya deleted the symfony-4-support branch September 17, 2018 12:09
@chirimoya
Copy link
Collaborator Author

@dantleech I guess we need to ;-) Can you add @wachterjohannes and @alexander-schranz as maintainer and maybe you find time to tag an new release. That would be awesome.

@dantleech
Copy link
Owner

dantleech commented Sep 18, 2018

Does this bundle still support 7.0? if so we should add that to travis, if not we should add it as a minimum requirement in composer.json

@dantleech
Copy link
Owner

dantleech commented Sep 18, 2018

Added @wachterjohannes and @alexander-schranz as collaborators (@chirimoya already has push access)

<argument type="service" id="phpcr_migrations.version_storage" />
<argument type="service" id="phpcr_migrations.version_finder" />
<tag name="console.command" />
</service>

<service id="phpcr_migrations.command.migrate" class="DTL\Bundle\PhpcrMigrations\Command\MigrateCommand">
<service id="phpcr_migrations.command.migrate" class="DTL\Bundle\PhpcrMigrations\Command\MigrateCommand" public="true">
Copy link
Owner

Choose a reason for hiding this comment

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

does this break BC at all?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no, but for symfony 4.0 you need to define the public services

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants