Skip to content
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

Upload Elevated Runner Script Just Once #3

Merged
merged 1 commit into from
Mar 2, 2016

Conversation

jerryk55
Copy link
Contributor

@jerryk55 jerryk55 commented Mar 1, 2016

Only upload the wrapper shell winrm-elevated-shell.ps1 once per connection.
No need to do so on every command request.

@sneal I broke the previous pull request into two parts as suggested. This change is relevant no matter the gem version.

Please review and merge when appropriate.

Only upload the wrapper shell winrm-elevated-shell.ps1 once per connection.
No need to do so on every command request.
@jerryk55 jerryk55 mentioned this pull request Mar 1, 2016
@mwrock
Copy link
Member

mwrock commented Mar 2, 2016

LGTM 👍

mwrock added a commit that referenced this pull request Mar 2, 2016
@mwrock mwrock merged commit 74eb712 into WinRb:master Mar 2, 2016
@jerryk55
Copy link
Contributor Author

jerryk55 commented Mar 2, 2016

Thanks, Matt!

@sneal sneal mentioned this pull request Mar 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants