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

add external to apply and default requirement type #196

Merged
merged 1 commit into from
Apr 18, 2022
Merged

Conversation

mike0sv
Copy link
Contributor

@mike0sv mike0sv commented Apr 14, 2022

No description provided.

@mike0sv mike0sv requested a review from a team April 14, 2022 14:08
@codecov
Copy link

codecov bot commented Apr 14, 2022

Codecov Report

Merging #196 (0adc3ba) into main (d5525e8) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #196      +/-   ##
==========================================
- Coverage   89.55%   89.51%   -0.04%     
==========================================
  Files          74       74              
  Lines        4911     4912       +1     
==========================================
- Hits         4398     4397       -1     
- Misses        513      515       +2     
Impacted Files Coverage Δ
mlem/cli/apply.py 95.23% <ø> (ø)
mlem/api/commands.py 88.88% <100.00%> (ø)
mlem/core/requirements.py 95.54% <100.00%> (+0.01%) ⬆️
mlem/contrib/heroku/meta.py 82.85% <0.00%> (-2.86%) ⬇️

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 d5525e8...0adc3ba. 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.

:shipit: :shipit: :shipit:

@aguschin aguschin merged commit e8f10af into main Apr 18, 2022
@aguschin aguschin deleted the fix/tutorial branch April 18, 2022 17:34
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants