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

PHP 8.0 support #38

Open
quazardous opened this issue Jun 1, 2021 · 5 comments
Open

PHP 8.0 support #38

quazardous opened this issue Jun 1, 2021 · 5 comments

Comments

@quazardous
Copy link

Hi,

not working for PHP 8

best regards

@tijiene
Copy link

tijiene commented Jun 14, 2021

Hello,
It's fixed here
master...emilZelty:php-8-compatibility

Please merge

Thank you

@Makapashev
Copy link

Makapashev commented Feb 22, 2022

Please adapt this project for PHP 8.* and Symfony 6

@spantaleev
Copy link

@Makapashev, I've forked both this and mongodb-migrations. See devture/mongodb-migrations and devture/mongodb-migrations-bundle.

You should be able to replace doesntmatter with devture in your composer.json and bump the versions to 4.0.* and it should work on PHP 8 / Symfony 6. Example composer.json:

{
	"requires": {
		"devture/mongodb-migrations": "4.0.*",
		"devture/mongodb-migrations-bundle": "4.0.*"
	}
}

I have no interest in maintaining these libraries or developing new features, but I wanted to migrate some systems to Symfony 6 and couldn't find any fork which worked on Symfony 6, so I had to fix things up a bit myself. I've tested generating migrations, migrating and rolling migrations back, so the main things seem to work.

@Makapashev
Copy link

@spantaleev Thank you so much!

@sterrien
Copy link

@caciobanu it seems you do not maintain anymore this repository, do you ?

When I see :

antimattr/mongodb-migrations ==> doesntmattr/mongodb-migrations ==> devture/mongodb-migrations

antimattr/mongodb-migrations-bundle ==> doesntmattr/mongodb-migrations-bundle ==> devture/mongodb-migrations-bundle

I think that something goes not pretty well.

But I understand that priorities change !

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

No branches or pull requests

5 participants