mia-platform / mlp

cli for easier deployment of Mia-Platform Console projects
https://docs.mia-platform.eu/docs/runtime_suite_tools/mlp/overview
Apache License 2.0
12 stars 2 forks source link

fix: awaitable resources start time comparison #60

Closed mattia-fumo closed 2 years ago

mattia-fumo commented 2 years ago

Fixed a bug in awaitable resources that caused some events to be skipped because time with different precision were wrongly compared

codecov-commenter commented 2 years ago

Codecov Report

Base: 67.92% // Head: 67.78% // Decreases project coverage by -0.14% :warning:

Coverage data is based on head (3d0500d) compared to base (88e6f72). Patch coverage: 57.14% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #60 +/- ## ========================================== - Coverage 67.92% 67.78% -0.15% ========================================== Files 10 10 Lines 1400 1403 +3 ========================================== Hits 951 951 - Misses 354 356 +2 - Partials 95 96 +1 ``` | [Impacted Files](https://codecov.io/gh/mia-platform/mlp/pull/60?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mia-Platform) | Coverage Δ | | |---|---|---| | [pkg/deploy/apply.go](https://codecov.io/gh/mia-platform/mlp/pull/60/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mia-Platform#diff-cGtnL2RlcGxveS9hcHBseS5nbw==) | `65.65% <57.14%> (-0.51%)` | :arrow_down: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mia-Platform). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mia-Platform)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.