8000 merge #5843 to 6.0.1 by TravisEz13 · Pull Request #5997 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]
Skip to content

merge #5843 to 6.0.1 #5997

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 24, 2018
Merged

merge #5843 to 6.0.1 #5997

merged 1 commit into from
Jan 24, 2018

Conversation

TravisEz13
Copy link
Member

merge #5843 to 6.0.1

@TravisEz13
tests: fix function to test for docker OS due to change to use linuxk…
b003a50 8000 
…it for mac (PowerShell#5843)

* Also, fix syntax issues to allow to work with released Pester
anmenaga
anmenaga approved these changes Jan 23, 2018
View reviewed changes
Copy link
@anmenaga anmenaga left a comment

Choose a reason for hiding this comment

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

LGTM.
This is needed for 6.0.1 container test verifications.

@TravisEz13
Copy link
Member Author

restarted windows due to a web listener timeout

@TravisEz13 TravisEz13 merged commit e49a81f into PowerShell:6.0.1 Jan 24, 2018
@TravisEz13 TravisEz13 deleted the fix_docker_tests branch January 24, 2018 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daxian-dbw daxian-dbw daxian-dbw approved these changes

@anmenaga anmenaga anmenaga approved these changes

@adityapatwardhan adityapatwardhan Awaiting requested review from adityapatwardhan

@SteveL-MSFT SteveL-MSFT Awaiting requested review from SteveL-MSFT

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TravisEz13 @daxian-dbw @anmenaga
0