Deploy mdbook only if owner is comtrya. #466
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #466 +/- ##
==========================================
- Coverage 28.59% 26.42% -2.17%
==========================================
Files 85 83 -2
Lines 2301 2312 +11
==========================================
- Hits 658 611 -47
- Misses 1643 1701 +58 ☔ View full report in Codecov by Sentry. |
6208e33 to
32c3573
Compare
32c3573 to
037094a
Compare
|
I definitely overlooked the pipeline running in forks causing an annoying failure. Thanks for the PR to fix that! |
|
Looks like there is an issue with it. I tried messing with the dependencies to see if maybe the upload step needed to depend on the build step and deploy step depend on the upload, but that wasn't quiet it either. |
I'm submitting a
What is the current behaviour?
The deploy mdbook workflow fails in my fork because I don't have permission to publish.
It would be great to see that all tests pass / fail when looking at my fork commit history without having to account for the deploy workflow always failing.