Skip to content

Only skip @github-api tests when GITHUB_TOKEN isn't present#4481

Merged
danielbachhuber merged 4 commits intomasterfrom
dont-skip-github-token
Nov 2, 2017
Merged

Only skip @github-api tests when GITHUB_TOKEN isn't present#4481
danielbachhuber merged 4 commits intomasterfrom
dont-skip-github-token

Conversation

@danielbachhuber
Copy link
Member

@danielbachhuber danielbachhuber added this to the 1.5.0 milestone Nov 2, 2017
@danielbachhuber danielbachhuber added the scope:testing Related to testing label Nov 2, 2017
@danielbachhuber danielbachhuber requested a review from a team November 2, 2017 14:39
@gitlost
Copy link
Contributor

gitlost commented Nov 2, 2017

The PHP 5.3 issue is to do with it not having a PHP_BINARY define I think, so WP_CLI::get_php_binary() needs to be updated to fall back to PHP_BINDIR as well - will push a fix for it...

@danielbachhuber
Copy link
Member Author

@gitlost Cool, thanks

@gitlost
Copy link
Contributor

gitlost commented Nov 2, 2017

Oh just noticed that FeatureContext has its own version of get_php_binary() - that should probably be updated as well...

@gitlost
Copy link
Contributor

gitlost commented Nov 2, 2017

Ha and also be made static...

Copy link
Contributor

@gitlost gitlost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll leave it to you to merge if you're ok with the further pushes.

(Note will PR some small speed-ups soon re the phar tests in particular which will claw back some of the slow-down in the test runtimes.)

@danielbachhuber danielbachhuber merged commit a8891f5 into master Nov 2, 2017
@danielbachhuber danielbachhuber deleted the dont-skip-github-token branch November 2, 2017 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants