Re: Can someone with Zend karma please merge PR #674?
| From: | Levi Morrison | Date: | Mon, 09 Jun 2014 00:05:50 +0000 |
| Subject: | Re: Can someone with Zend karma please merge PR #674? | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-74787@lists.php.net to get a copy of this message | ||
On Sun, Jun 8, 2014 at 5:57 PM, Stas Malyshev <smalyshev@sugarcrm.com> wrote:
> Hi!
>
>> https://github.com/php/php-src/pull/674
>>
>> It fixes bug 66622: https://bugs.php.net/bug.php?id=66622
>> I've prodded people on #php.pecl and nobody has had any complaints
>
> Didn't we just has a discussion about it an hour ago?
No, you mentioned one tiny thing that isn't really about the bug fix.
That's hardly a conversation.
>> about it, so I don't anticipate any issues; if there is an issue do
>> let me know.
>
> I still don't think we should move the tests around.
I disagree but it is a minor point. Others have said it makes sense to
put related tests in a folder in IRC.
I don't really care; you can easily not apply those changes.