icesat2py / icepyx

Python tools for obtaining and working with ICESat-2 data
https://icepyx.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
219 stars 107 forks source link

trigger to unit tests upon completion of UML action #619

Closed JessicaS11 closed 1 month ago

JessicaS11 commented 1 month ago

Addresses #618 by explicitly triggering the required unit tests after the UML action is run.

github-actions[bot] commented 1 month ago

Binder :point_left: Launch a binder notebook on this branch for commit 8f27965e42c5b28fb3513652edc2149f2b2d5019

I will automatically update this comment whenever this PR is modified

Binder :point_left: Launch a binder notebook on this branch for commit c8084a71ab0dbd1c37566f0f40b54a6d386aa8a2

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 66.79%. Comparing base (628ad8f) to head (c8084a7). Report is 1 commits behind head on development.

:exclamation: There is a different number of reports uploaded between BASE (628ad8f) and HEAD (c8084a7). Click for more details.

HEAD has 1 upload less than BASE | Flag | BASE (628ad8f) | HEAD (c8084a7) | |------|------|------| ||3|2|
Additional details and impacted files ```diff @@ Coverage Diff @@ ## development #619 +/- ## =============================================== - Coverage 71.81% 66.79% -5.02% =============================================== Files 38 36 -2 Lines 3136 3078 -58 Branches 599 593 -6 =============================================== - Hits 2252 2056 -196 - Misses 773 933 +160 + Partials 111 89 -22 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

JessicaS11 commented 1 month ago

(i.e. will it mark the checks completed when the triggering event is another workflow?)! But let's give it a shot ;)

It appears to have - the UML diagrams test ran and both of the unit tests are now successfully reporting statuses.

image
mfisher87 commented 1 month ago

:raised_hands: Nice!!