neo4j-php-client icon indicating copy to clipboard operation
neo4j-php-client copied to clipboard

update psr/http-message ^2.0 & add ext-mbstring & update stefanak-michal/bolt ^5.1 -> ^6.0

Open lcp0578 opened this issue 1 year ago • 6 comments

update psr/http-message ^2.0

lcp0578 avatar Feb 23 '24 06:02 lcp0578

When I use it for symfony 6.4 it need psr/http-message ^2.0

lcp0578 avatar Feb 23 '24 06:02 lcp0578

Thank you for the update! I'll be running some test and merge it later today. Good job!

transistive avatar Feb 23 '24 09:02 transistive

I'm not sureneed to add constraints,

"ext-mbstring": "*"

although Blot already has constraints, https://github.com/neo4j-php/Bolt/blob/master/composer.json#L12 but when I require this libraries, composer didn't tell me that I needed mbstring extensions, which really wasted a lot of my time, and I found out through debugging that I really didn't have it, but it's not so intuitive, and I hope it will reduce the time of other people who use this library.

lcp0578 avatar Feb 23 '24 11:02 lcp0578

I think "php": "^8.0", https://github.com/neo4j-php/neo4j-php-client/blob/main/composer.json#L24 so,
image

https://github.com/neo4j-php/Bolt/releases/tag/v6.0.0

I'd like PHP's version constraints to be consistent.

lcp0578 avatar Feb 23 '24 11:02 lcp0578

If you are updating more than psr/http-message you should update PR name.

stefanak-michal avatar Feb 23 '24 11:02 stefanak-michal

If you are updating more than psr/http-message you should update PR name.

I update the PR name, thx.

lcp0578 avatar Feb 23 '24 11:02 lcp0578

@transistive If you have time to deal with this PR, my team and I urgently need to update a version.

lcp0578 avatar Feb 26 '24 07:02 lcp0578