Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Plugins error document origin #289

Merged
merged 2 commits into from
Jul 13, 2021
Merged

Plugins error document origin #289

merged 2 commits into from
Jul 13, 2021

Conversation

wild-endeavor
Copy link
Contributor

Signed-off-by: wild-endeavor [email protected]

TL;DR

flyteorg/flyteplugins#188

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

Bump plugins

Signed-off-by: wild-endeavor <[email protected]>
@codecov
Copy link

codecov bot commented Jul 12, 2021

Codecov Report

Merging #289 (500f007) into master (d91dd6b) will increase coverage by 0.61%.
The diff coverage is n/a.

❗ Current head 500f007 differs from pull request most recent head a36dfb2. Consider uploading reports for the commit a36dfb2 to get more accurate results

@@ -234,7 +234,10 @@ github.com/flyteorg/flyteidl v0.19.5 h1:qNhNK6mhCTuOms7zJmBtog6bLQJhBj+iScf1IlHd
github.com/flyteorg/flyteidl v0.19.5/go.mod h1:576W2ViEyjTpT+kEVHAGbrTP3HARNUZ/eCwrNPmdx9U=
github.com/flyteorg/flyteplugins v0.5.56 h1:LF/dwMFJDSMEmOp8hd9rU4Et4oyn0K+LgMzcHOu/xrw=
github.com/flyteorg/flyteplugins v0.5.56/go.mod h1:Jp5WheQMI08luZmgcmcgyjtzakKH0tPws/t35DzpKUA=
github.com/flyteorg/flyteplugins v0.5.59 h1:Uw1xlrlx5rSTpdTMwJTo7mbqHI7X7p7CFVm3473iRjo=
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you run go mod tidy plz

Signed-off-by: wild-endeavor <[email protected]>
@wild-endeavor
Copy link
Contributor Author

done.

@EngHabu EngHabu merged commit 74a042d into master Jul 13, 2021
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
* bump

Signed-off-by: wild-endeavor <[email protected]>

* tidy

Signed-off-by: wild-endeavor <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants