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

[wip]: support Windows #16

Closed
wants to merge 4 commits into from
Closed

Conversation

nomyfan
Copy link

@nomyfan nomyfan commented Apr 4, 2021

Hi, I made some efforts to make it run on Windows successfully and to dome refactoring.

But I have trouble with setting up the GitHub actions for Windows currently.

@nomyfan
Copy link
Author

nomyfan commented Apr 4, 2021

#15

@jvns
Copy link
Owner

jvns commented Apr 4, 2021

Can we avoid refactoring in this PR? I'd rather make a minimal change to support Windows here -- it's hard for me to review the PR when so many lines have changed :). Thanks!

@nomyfan
Copy link
Author

nomyfan commented Apr 5, 2021

Well, I opened another PR #17

@nomyfan nomyfan closed this Apr 5, 2021
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