flytekit icon indicating copy to clipboard operation
flytekit copied to clipboard

Separate executor docker image from plugin code

Open EngHabu opened this issue 4 years ago • 1 comments

Signed-off-by: Haytham Abuelfutuh [email protected]

TL;DR

Please replace this text with a description of what this PR accomplishes.

Type

  • [ ] Bug Fix
  • [ ] Feature
  • [ ] Plugin

Are all requirements met?

  • [ ] Code completed
  • [ ] Smoke tested
  • [ ] Unit tests added
  • [ ] Code documentation added
  • [ ] Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

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

Follow-up issue

NA OR https://github.com/lyft/flyte/issues/

EngHabu avatar Jul 15 '21 16:07 EngHabu

Codecov Report

Merging #554 (b6c4f6b) into master (6875a5f) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #554   +/-   ##
=======================================
  Coverage   85.49%   85.49%           
=======================================
  Files         372      372           
  Lines       28976    28976           
  Branches     2318     2318           
=======================================
  Hits        24774    24774           
  Misses       3559     3559           
  Partials      643      643           

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 6875a5f...b6c4f6b. Read the comment docs.

codecov[bot] avatar Jul 15 '21 16:07 codecov[bot]