On 07.02.2019 at 16:14, Sjon Hortensius wrote: > On Wed, Feb 6, 2019 at 4:14 PM Ben Ramsey <b...@benramsey.com> wrote: > >>> On Feb 6, 2019, at 01:22, Peter Kokot <peterko...@gmail.com> wrote: >>> I can help sort this mess. A separate fork out of the php/php-src was >>> not such a good idea. >> >> Sorry. I made many poor decisions around how I managed that entire event. >> It’s my own personal Fyre Festival. :-( >> >> I’m happy to help in any way I can to get these ready to merge into >> php-src. Let me know how I can help. > > anyone with access to the php-src git repo should be able to merge them > pretty easily; it's a matter of running: > > curl -s ' > https://github.com/phpcommunity/phptestfest-php-src/pulls?page=1&q=is%3Apr+is%3Aopen+review%3Aapproved'|grep > -Po '(?<=<div id="issue_)[0-9]*'|while read id > do > curl > https://patch-diff.githubusercontent.com/raw/phpcommunity/phptestfest-php-src/pull/$id.patch|git > am > done > > and increase the page-number in the first URL to includes pages 2 & 3 as > well. Run this in your checkout (possibly on a branch) and push that to > github
It seems to me that the tests should be reviewed (again). The first one I've seen[1], should be removed, since it tests general ZPP behavior, and we're in the process to remove such tests generally[2]. [1] <https://github.com/phpcommunity/phptestfest-php-src/pull/155/files#diff-33e85609b0c12ee2d4b1c5f6190fdfa4> [2] <https://github.com/php/php-src/pull/3783> -- Christoph M. Becker -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
