Garden-AI / garden

https://garden-ai.readthedocs.io
MIT License
20 stars 4 forks source link

Make sure we don't publish entrypoints with incompatible dill versions #368

Closed WillEngler closed 10 months ago

WillEngler commented 10 months ago

Fixes #357.

Overview

Testing

I confirmed that raising the dill error works as suspected for a notebook with and without the right version of dill.

Documentation

None


:books: Documentation preview :books:: https://garden-ai--368.org.readthedocs.build/en/368/

codecov-commenter commented 10 months ago

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (bb7f1a0) 76.79% compared to head (1a09f3b) 76.62%. Report is 1 commits behind head on main.

Files Patch % Lines
garden_ai/scripts/save_session_and_metadata.py 0.00% 5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #368 +/- ## ========================================== - Coverage 76.79% 76.62% -0.17% ========================================== Files 42 42 Lines 2275 2280 +5 ========================================== Hits 1747 1747 - Misses 528 533 +5 ```

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