Skip to content

Update requirements#809

Closed
pingsutw wants to merge 4 commits into
masterfrom
test1
Closed

Update requirements#809
pingsutw wants to merge 4 commits into
masterfrom
test1

Conversation

@pingsutw

Copy link
Copy Markdown
Member

Signed-off-by: Kevin Su pingsutw@apache.org

TL;DR

Fix the GA's errors
https://github.com/flyteorg/flytekit/actions/runs/1680435426

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

Signed-off-by: Kevin Su <pingsutw@apache.org>
@codecov

codecov Bot commented Jan 11, 2022

Copy link
Copy Markdown

Codecov Report

Merging #809 (e5bea35) into master (3296c1c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #809   +/-   ##
=======================================
  Coverage   85.82%   85.82%           
=======================================
  Files         346      346           
  Lines       29801    29801           
  Branches     3597     3597           
=======================================
  Hits        25576    25576           
  Misses       3572     3572           
  Partials      653      653           

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 3296c1c...e5bea35. Read the comment docs.

Signed-off-by: Kevin Su <pingsutw@apache.org>
Signed-off-by: Kevin Su <pingsutw@apache.org>
Signed-off-by: Kevin Su <pingsutw@apache.org>
@eapolinario

Copy link
Copy Markdown
Collaborator

Superseded by #810.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants