New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass WP_CLI_STRICT_ARGS_MODE on to ssh if set. #4207

Merged
merged 2 commits into from Jul 10, 2017

Conversation

3 participants
@gitlost
Contributor

gitlost commented Jul 8, 2017

Fixes #4206

Prefixes ssh command with WP_CLI_STRICT_ARGS_MODE=1 if set locally.

@gitlost gitlost requested review from wp-cli/committers and removed request for wp-cli/committers Jul 8, 2017

@gitlost

This comment has been minimized.

Show comment
Hide comment
@gitlost

gitlost Jul 8, 2017

Contributor

Okay that doesn't work.

Contributor

gitlost commented Jul 8, 2017

Okay that doesn't work.

@gitlost gitlost requested a review from wp-cli/committers Jul 8, 2017

@danielbachhuber danielbachhuber added this to the 1.3.0 milestone Jul 10, 2017

@danielbachhuber danielbachhuber merged commit a9c94e6 into master Jul 10, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@danielbachhuber danielbachhuber deleted the issue_4206 branch Jul 10, 2017

@danielbachhuber danielbachhuber added the bug label Jul 10, 2017

@paulgibbs

This comment has been minimized.

Show comment
Hide comment
@paulgibbs

paulgibbs Jul 10, 2017

Thanks for getting this fixed promptly!

paulgibbs commented Jul 10, 2017

Thanks for getting this fixed promptly!

@danielbachhuber

This comment has been minimized.

Show comment
Hide comment
@danielbachhuber

danielbachhuber Jul 10, 2017

Member

You're welcome!

Member

danielbachhuber commented Jul 10, 2017

You're welcome!

@paulgibbs paulgibbs referenced this pull request Aug 21, 2017

Merged

Use WordHat Directly #13

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