Michael Osipov

Results 569 comments of Michael Osipov

Made progress: There is no usable Horde_Text_Diff on packagist. I had to fork it because it not PHP 8 compatible. [@bytestream](https://github.com/bytestream) has forked a lot of the Horde modules, but...

I have intentionally switched to the `Native` diff impl in Text_Diff because the way `Xdiff` is used is that the context is always applied to the entire file and the...

One note on PEAR: I wasn't able to make it run with PHP 8.2, it just crashes and Horde_Text_Diff isn't available in a newer version anyway, I am considering to...

@bytestream I did indeed check Util yesterday already and happily used your approach. Kudos for that. All tests pass with PHP 8.2-beta2. From your point of view it does not...

@ralflang Looking into your fork...

@ralflang Here is the diff: https://github.com/maintaina-com/Text_Diff/compare/FRAMEWORK_6_0...websvnphp:Text_Diff:php-8.x I would start to create PRs against your fork, if you agree.

Tests also look good for me on my fork: ``` # ./vendor/bin/phpunit PHPUnit 9.5.24 #StandWithUkraine ...I....I...R... 16 / 16 (100%) Time: 00:00.019, Memory: 6.00 MB There was 1 risky test:...

> > But on your fork it does not even run: > > Give me a day or two to care about the basics. Your PR's are much appreciated. >...

@ralflang I have now created all of the PRs to align the code. As soon as those are approved, I will port from our fork to your fork by using...

@ralflang Thank you for helping out so quick. I have a tentative version working with `FRAMEWORK_6_0` branch. Will give you an update next week. Switched from composer to PEAR. One...