flytekit icon indicating copy to clipboard operation
flytekit copied to clipboard

Prettyfying and making it cross platform

Open kumare3 opened this issue 2 years ago • 1 comments

Tracking issue

https://github.com/flyteorg/flyte/issues/

Why are the changes needed?

What changes were proposed in this pull request?

How was this patch tested?

Setup process

Screenshots

Check all the applicable boxes

  • [ ] I updated the documentation accordingly.
  • [ ] All new and existing tests passed.
  • [ ] All commits are signed-off.

Related PRs

Docs link

kumare3 avatar Jan 02 '24 23:01 kumare3

Codecov Report

Attention: Patch coverage is 48.83721% with 44 lines in your changes are missing coverage. Please review.

Project coverage is 53.62%. Comparing base (b475c87) to head (601b8d8).

Files Patch % Lines
flytekit/tools/repo.py 33.33% 20 Missing :warning:
flytekit/tools/serialize_helpers.py 5.00% 19 Missing :warning:
flytekit/loggers.py 86.11% 1 Missing and 4 partials :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2083       +/-   ##
===========================================
- Coverage   83.05%   53.62%   -29.44%     
===========================================
  Files         324      180      -144     
  Lines       24860    18150     -6710     
  Branches     3547     3555        +8     
===========================================
- Hits        20648     9733    -10915     
- Misses       3589     7987     +4398     
+ Partials      623      430      -193     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jan 02 '24 23:01 codecov[bot]