-
Notifications
You must be signed in to change notification settings - Fork 87
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
Fix windows unit test CI #1490
Fix windows unit test CI #1490
Conversation
@@ -1,7 +1,7 @@ | |||
version: 2.1 | |||
|
|||
orbs: | |||
windows: circleci/windows@2.2.0 | |||
windows: circleci/windows@2.4.0 |
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.
This wasn't needed for this PR, I just figured we should use latest.
Codecov Report
@@ Coverage Diff @@
## main #1490 +/- ##
=======================================
Coverage 100.0% 100.0%
=======================================
Files 223 223
Lines 15100 15100
=======================================
Hits 15093 15093
Misses 7 7 Continue to review full report at Codecov.
|
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.
@dsherry Thanks for the fix! The docs change looks great to me.
Thanks @freddyaboulton for the tips in slack! He mentioned:
My suggested fix: install
numba
using conda, and mention this on the install page in the "Windows" section.