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

Remove binary prefix and cast from unrelated tests#2194

Closed
pmmaga wants to merge 2 commits into
php:masterphp/php-src:masterfrom
pmmaga:remove-binary-unrelated-testspmmaga/php-src:remove-binary-unrelated-testsCopy head branch name to clipboard
Closed

Remove binary prefix and cast from unrelated tests#2194
pmmaga wants to merge 2 commits into
php:masterphp/php-src:masterfrom
pmmaga:remove-binary-unrelated-testspmmaga/php-src:remove-binary-unrelated-testsCopy head branch name to clipboard

Conversation

@pmmaga
Copy link
Copy Markdown
Contributor

@pmmaga pmmaga commented Nov 8, 2016

As a child PR from #2187, this aims only at removing the binary prefix and cast from tests that are not directly related to binary string handling.

@php-pulls
Copy link
Copy Markdown

Comment on behalf of krakjoe at php.net:

adding label

@pmmaga
Copy link
Copy Markdown
Contributor Author

pmmaga commented Nov 20, 2016

The failing test on Travis is unrelated to this changes

@nikic
Copy link
Copy Markdown
Member

nikic commented Nov 20, 2016

Merge of #2209 caused some conflicts. Can you please rebase this PR?

@pmmaga pmmaga force-pushed the remove-binary-unrelated-tests branch from bfba8f6 to bb76bc9 Compare November 20, 2016 17:38
@pmmaga pmmaga force-pushed the remove-binary-unrelated-tests branch from bb76bc9 to 524943f Compare November 20, 2016 18:24
@nikic
Copy link
Copy Markdown
Member

nikic commented Nov 20, 2016

Merged via 9c5af4e and 617dcf1. Thanks!

@nikic nikic closed this Nov 20, 2016
@kelunik
Copy link
Copy Markdown
Member

kelunik commented Nov 25, 2016

Any reason why this has been merged into master only and not 7.1?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

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