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

Fix flyctl reuse app #674

Merged
merged 2 commits into from
May 11, 2023
Merged

Fix flyctl reuse app #674

merged 2 commits into from
May 11, 2023

Conversation

aminalaee
Copy link
Contributor

Closes #667

@aminalaee aminalaee requested a review from a team May 11, 2023 11:26
@aminalaee aminalaee temporarily deployed to internal May 11, 2023 11:26 — with GitHub Actions Inactive
@aminalaee aminalaee temporarily deployed to internal May 11, 2023 11:27 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented May 11, 2023

Codecov Report

Patch coverage: 28.57% and project coverage change: -0.10 ⚠️

Comparison is base (2b63b34) 85.75% compared to head (e9bdf91) 85.66%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #674      +/-   ##
==========================================
- Coverage   85.75%   85.66%   -0.10%     
==========================================
  Files         109      109              
  Lines        9922     9928       +6     
==========================================
- Hits         8509     8505       -4     
- Misses       1413     1423      +10     
Impacted Files Coverage Δ
mlem/contrib/pandas.py 92.45% <25.00%> (-0.77%) ⬇️
mlem/contrib/flyio/meta.py 45.29% <33.33%> (-0.79%) ⬇️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@aguschin aguschin merged commit 8bf5e13 into main May 11, 2023
@aguschin aguschin deleted the fix-flyctl-reuse-app branch May 11, 2023 12:04
# 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.

Confusing error in deploy run flyio when app exists
2 participants