-
Notifications
You must be signed in to change notification settings - Fork 32
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
Install dev wheels properly #222
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #222 +/- ##
=======================================
Coverage 84.31% 84.31%
=======================================
Files 35 35
Lines 6452 6452
=======================================
Hits 5440 5440
Misses 1012 1012 ☔ View full report in Codecov by Sentry. |
f1bd2dd
to
9f6bb55
Compare
9f6bb55
to
6962cc9
Compare
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 looks good to me, for what it's worth
I added a commit due to: |
But do you actually use devdeps? I don't see it in the jobs here. |
#213 adds devdeps as an optional workflow when adding the |
OK, then maybe you can merge this first and see if this setup really works on your other PR. 🤞 Or feel free to cherry pick the commits over and close this without merge. |
You can simplify devdeps calls and you can also install astropy dev from wheel.