Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Test Round: Added tests to check operation with existing implementation#13399

Merged
Girgias merged 1 commit into
php:masterphp/php-src:masterfrom
SakiTakamachi:feature/expand-rounding-target-testSakiTakamachi/php-src:feature/expand-rounding-target-testCopy head branch name to clipboard
Feb 17, 2024
Merged

Test Round: Added tests to check operation with existing implementation#13399
Girgias merged 1 commit into
php:masterphp/php-src:masterfrom
SakiTakamachi:feature/expand-rounding-target-testSakiTakamachi/php-src:feature/expand-rounding-target-testCopy head branch name to clipboard

Conversation

@SakiTakamachi
Copy link
Copy Markdown
Member

Related #12222

@Girgias
Copy link
Copy Markdown
Member

Girgias commented Feb 17, 2024

I didn't mean to say to create a new PR to add the tests, but just to split your existing PR into two commits :)

But this is fine too :)

@Girgias Girgias merged commit ed1c9d8 into php:master Feb 17, 2024
@SakiTakamachi
Copy link
Copy Markdown
Member Author

I see, I misunderstood. Thank you!

@SakiTakamachi SakiTakamachi deleted the feature/expand-rounding-target-test branch February 17, 2024 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.