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

test on windows #398

Merged
merged 41 commits into from
Dec 26, 2019
Merged

test on windows #398

merged 41 commits into from
Dec 26, 2019

Conversation

kavigupta
Copy link
Contributor

No description provided.

@kavigupta
Copy link
Contributor Author

oh there's actually no support for python on these systems 😿. I'll see if I can find a workaround, but for now let's continue manual smoke testing.

@mehrdadn mehrdadn self-requested a review December 26, 2019 03:06
@mehrdadn
Copy link
Contributor

@kavigupta ping me when the PR is done? I might need to make some changes.

python: "3.7"
language: sh
before_install:
- choco install python --version=3.7.2 --params "/InstallDir:C:\\Python"
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor, but you don't actually need the double-backslash. Backslashes stay as-is unless followed by one of very few characters (which I think is just dollar sign, quotes, or sometimes an exclamation point).

@kavigupta kavigupta changed the title test on all 3 os test on windows Dec 26, 2019
@kavigupta kavigupta merged commit 89084bf into master Dec 26, 2019
@kavigupta kavigupta deleted the multi-os-tests branch December 26, 2019 07:52
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