Skip to content

Conversation

@Chris53897
Copy link
Contributor

@Chris53897 Chris53897 commented Dec 8, 2025

allow laravel/framework > 11
13.0.x-dev needs to be reverted after a stable release is available.

#242

I would suggest to bump ^6.0|^7.0|^8.0 to ^6.4|^7.0|^8.0 if maintainers agree.

removed symfony/password-hasher but this needs a closer look
@pounard maybe you can have a look if this conflict can be removed.
https://github.com/makinacorpus/DbToolsBundle/pull/152/files#diff-d2ab9925cad7eac58e0ff4cc0d251a937ecf49e4b6bf57f8b95aab76648a9d34R45

allow laravel/framework > 11
allow laravel/framework > 11
allow laravel/framework > 11
@jdecool
Copy link
Contributor

jdecool commented Jan 3, 2026

I'm interested in this PR.

Are you planning to finish it ? Do you need help ?

@Chris53897
Copy link
Contributor Author

There is only one ci check failing because of ignored errors.
I still need feedback from the maintainer
If you want you can have a look at the removed conflict in composer.json see link above

@SimonMellerin
Copy link
Member

Sorry I missed the notification, I'll try to check this PR this week

@SimonMellerin SimonMellerin self-requested a review January 21, 2026 16:37
@SimonMellerin
Copy link
Member

SimonMellerin commented Jan 22, 2026

@Chris53897 , can you rebase your branch on main to fix these phpstan errors ?

Thanks !

@pounard
Copy link
Member

pounard commented Jan 22, 2026

That's a really usefull PR, thanks, nevertheless I'd be more comfortable testing it in a real project (even some dumb mockup) on our side prior merging it.

I intend to refactor the whole local docker stack and do some fixes in the github ci configuration in the next few days, I will also check that static analysis is correctly run on lowest and highest dependencies. I'm all in merging this, but after those refactors have been done.

Thanks for your time and patience!

@Chris53897 Chris53897 closed this Jan 23, 2026
@Chris53897 Chris53897 reopened this Jan 23, 2026
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.

5 participants