CI Skip publishing workflow on fork#559
Conversation
| @@ -1,4 +1,4 @@ | |||
| name: Publish Python 🐍 distribution 📦 to PyPI and TestPyPI | |||
| name: Publish cloudpickle 🥒 distribution 📦 to PyPI and TestPyPI | |||
There was a problem hiding this comment.
This workflow doesn't publish Python but cloudpickle :)
Result of a quick copy paste.
| with: | ||
| name: python-package-distributions | ||
| path: dist/ | ||
| retention-days: 1 |
There was a problem hiding this comment.
by defaults artifacts are kept 90 days. For this workflow we only need them during the workflow run and 1 day is the minimum
ogrisel
left a comment
ogrisel
left a comment
There was a problem hiding this comment.
Thanks for the PR and the pickle emoji in particular.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #559 +/- ##
===========================================
- Coverage 96.34% 84.73% -11.62%
===========================================
Files 3 2 -1
Lines 575 570 -5
Branches 121 102 -19
===========================================
- Hits 554 483 -71
- Misses 11 69 +58
- Partials 10 18 +8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
2 participants
This workflow should not run on forks and is annoying because github sends a notification each time you push to fork to tell you that it failed.