codecov-commenter
commented
1 month ago Codecov Report
Attention: Patch coverage is 15.00000% with 17 lines in your changes are missing coverage. Please review.
Project coverage is 76.31%. Comparing base (
d30a2b7) to head (c8a6be0). Report is 1 commits behind head on main.
| Files | Patch % | Lines |
|---|---|---|
| garden_ai/scripts/save_session_and_metadata.py | 11.11% | 16 Missing :warning: |
| garden_ai/app/garden.py | 0.00% | 1 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 #470 +/- ## ========================================== + Coverage 76.09% 76.31% +0.21% ========================================== Files 51 51 Lines 3301 3356 +55 ========================================== + Hits 2512 2561 +49 - Misses 789 795 +6 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Resolves #375
Overview
Pin requirements in the
RegisteredEntrypointmetadata if a requirements file is present when a notebook is published.Write the requirements into
metadata.jsoninside the published container image. This ensures when a garden is published to the search index the metadata for each entrypoint includes the requirements.Fix a small bug where creating a garden with no contributors threw an exception.
add flake8 to pre-commit
add emacs TAGS to
.gitignoreTesting
Tested manually by creating a notebook with a requirements file and a single entrypoint, publishing it, and confirming that the
metadata.jsonin the published image contains the requirements data. Tested another notebook with no added requirements to ensure other functionality is not impacted.I am struggling to come up with efficient unit tests for this. I want to confirm the
metadata.jsonis correct, but to do that I need to build a container image which will significantly slow down the test suite.Documentation
Updates the doc comments for
RegisteredEntrypoint📚 Documentation preview 📚: https://garden-ai--470.org.readthedocs.build/en/470/