Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Add missing docstrings and refactor a little #261

Merged
merged 4 commits into from
May 20, 2022
Merged

Conversation

mike0sv
Copy link
Contributor

@mike0sv mike0sv commented May 19, 2022

closes #70

@mike0sv mike0sv requested a review from a team May 19, 2022 12:56
@codecov
Copy link

codecov bot commented May 19, 2022

Codecov Report

Merging #261 (5498531) into main (83f169f) will decrease coverage by 0.06%.
The diff coverage is 91.00%.

@@            Coverage Diff             @@
##             main     #261      +/-   ##
==========================================
- Coverage   89.23%   89.17%   -0.07%     
==========================================
  Files          77       75       -2     
  Lines        5544     5568      +24     
==========================================
+ Hits         4947     4965      +18     
- Misses        597      603       +6     
Impacted Files Coverage Δ
mlem/config.py 97.53% <ø> (ø)
mlem/core/artifacts.py 99.40% <ø> (ø)
mlem/core/hooks.py 95.08% <ø> (ø)
mlem/core/import_objects.py 97.56% <ø> (ø)
mlem/core/index.py 67.32% <ø> (ø)
mlem/core/meta_io.py 93.92% <ø> (ø)
mlem/core/model.py 93.38% <ø> (ø)
mlem/core/requirements.py 95.05% <ø> (ø)
mlem/ext.py 86.45% <ø> (ø)
mlem/runtime/interface.py 84.78% <ø> (ø)
... and 33 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 83a17ca...5498531. Read the comment docs.

Copy link
Contributor

@aguschin aguschin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow. This is awesome!

@mike0sv mike0sv merged commit 575241b into main May 20, 2022
@mike0sv mike0sv deleted the feature/docstrings branch May 20, 2022 21:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extend docstrings coverage
2 participants