Re: Question about merged PR #937
| From: | guilhermeblanco@gmail.com | Date: | Thu, 19 Sep 2019 17:20:14 +0000 |
| Subject: | Re: Question about merged PR #937 | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-107235@lists.php.net to get a copy of this message | ||
Thanks for the clarification Christoph!
Somehow I couldn't see it when looking for the commit hash. Sorry for
the disturbance in the list here.
PS: I find it funny that even though I'm the original author of the
code, I don't show as a contributor in any statistics.
It'd be a shame to not be considered as a valid voter for "lack of
recent contribution" to the language.
Cheers,
On Thu, Sep 19, 2019 at 12:58 PM Christoph M. Becker <cmbecker69@gmx.de> wrote:
>
> On 19.09.2019 at 17:01, guilhermeblanco@gmail.com wrote:
> > One of my old PRs to PHP that was claimed to be merged disappeared
> > from master. However, the upgrade note is still there in master and
> > 7.4beta1.
> >
> > Here is the PR: https://github.com/php/php-src/pull/937
> > Here is the commit referencing it:
> >
> > https://github.com/php/php-src/commit/0adfa03397edcde8cba3bae2032b1f2ef26ea760
> >
> >
> > Please let me know if there's anything I can do to help.
>
> From what I can tell, everything is there. PR #911 has been merged as
>
> <http://git.php.net/?p=php-src.git;a=commit;h=094d409b3d34c51f49e0121e5ccfe8b2a717aaf6>,
> and parts of that PR already with PR #928 as
>
> <http://git.php.net/?p=php-src.git;a=commit;h=f48c2327403ce76a870e74f31a069a64dfb711a2>.
>
> --
> Christoph M. Becker
--
Guilherme Blanco
SVP Technology at Statflo Inc.
Mobile: +1 647 232 5599