-
-
Notifications
You must be signed in to change notification settings - Fork 750
Migrate to GitHub Actions and change install script to environment files #4218
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
Migrate to GitHub Actions and change install script to environment files #4218
Conversation
gforsyth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @jacobtomlinson -- do you want to merge from master? I think those windows tests should be passing now (although I also think they should've passed after you pinned pytest-asyncio, so...)
Co-authored-by: Gil Forsyth <[email protected]>
|
Thanks @gforsyth. I think we may still be blocked here on some SSH tests which are failing. I think GitHub Actions causes some problem with running our mock SSH server. |
Co-authored-by: jakirkham <[email protected]>
|
Pushed a commit to fix the merge conflict. Hope that is ok 🙂 |
|
Thanks @jakirkham ! |
|
Ofc! Do you have a sense of the current issues we are running into here? I looked at CI, but nothing obvious stuck out at me. |
|
Superseded by #4504; please close |
Closes #4214
This PR migrates CI to GitHub Actions.
Changes: