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

client: Pass all arguments to ExecuteCommandRequest #172

Merged
merged 1 commit into from Mar 2, 2017

Conversation

Projects
None yet
3 participants
@smarter
Contributor

smarter commented Feb 21, 2017

Previously, only the first argument was passed.

client: Pass all arguments to ExecuteCommandRequest
Previously, only the first argument was passed.
@msftclas

This comment has been minimized.

Show comment
Hide comment
@msftclas

msftclas Feb 21, 2017

Hi @smarter, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

msftclas commented Feb 21, 2017

Hi @smarter, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

@msftclas

This comment has been minimized.

Show comment
Hide comment
@msftclas

msftclas Feb 21, 2017

Hi @smarter, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

msftclas commented Feb 21, 2017

Hi @smarter, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

@dbaeumer

This comment has been minimized.

Show comment
Hide comment
@dbaeumer

dbaeumer Mar 2, 2017

Member

Good catch.

Member

dbaeumer commented Mar 2, 2017

Good catch.

@dbaeumer dbaeumer merged commit e0cfdc9 into Microsoft:master Mar 2, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment