-
Notifications
You must be signed in to change notification settings - Fork 1
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
DM-47128: PyPI deployment and entry points #21
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #21 +/- ##
==========================================
- Coverage 82.30% 82.29% -0.02%
==========================================
Files 16 18 +2
Lines 1074 1079 +5
Branches 276 174 -102
==========================================
+ Hits 884 888 +4
- Misses 165 166 +1
Partials 25 25 ☔ View full report in Codecov by Sentry. |
|
||
- name: Upload coverage to codecov | ||
uses: codecov/codecov-action@v4 | ||
with: | ||
files: ./coverage.xml | ||
token: ${{ secrets.CODECOV_TOKEN }} | ||
|
||
check-changes: |
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.
@mwittgen is the plan to put this code in a reusable workflow? It looks like we only made this change in sphgeom and not all the other middleware packages.
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.
Yes.
1bed268
to
fc7c7fa
Compare
fc7c7fa
to
8575456
Compare
The workflow should run on tag
|
Thanks for spotting the missing tags section. |
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 now. Quickly tested the upload check in my own fork.
No description provided.