Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:125089
Date: Wed, 21 Aug 2024 10:29:54 +0100
To: internals@lists.php.net, Marc Bennewitz <marc@mabe.berlin>,
 PHP internals <internals@lists.php.net>
Subject: Re: [PHP-DEV] DatePeriod microsecond support
User-Agent: K-9 Mail for Android
In-Reply-To: <10ddddb0-6bd0-4727-a793-4b0276fbd9bd@mabe.berlin>
References: <10ddddb0-6bd0-4727-a793-4b0276fbd9bd@mabe.berlin>
Message-ID: <E8CF0C73-9844-4A25-A2F4-35A6DE42E3B4@php.net>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain;
 charset=utf-8
Content-Transfer-Encoding: quoted-printable
From: derick@php.net (Derick Rethans)

On 21 August 2024 09:59:53 BST, Marc Bennewitz <marc@mabe=2Eberlin> wrote:
>Hi,
>
>The DatePeriod class works based on DateTimeInterface but does not take t=
he microsecond of the date-time instance into account [1]=2E
>To fix that I have opened a small PR [2] 2 months ago which got approved =
quickly as well but nothing more happened until then=2E
>
>
>I'm wring here to kindly ask if this can be merged to be included in 8=2E=
4 or if I should do something else?
>
>[1] https://github=2Ecom/php/php-src/issues/13773
>[2] https://github=2Ecom/php/php-src/pull/14581
>
>Thanks,
>Marc
>

I'll get this sorted=2E IMO this is also a bug fix, but as we are in featu=
re freeze I'll ask thr release managers=2E=20

cheers
Derick