Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@skriss
Copy link
Member

@skriss skriss commented Nov 30, 2020

Reverts #3159

Reverting this for now since it removed some publicly exported types from the API package which was a breaking API change for consumers of the Go package.

@skriss skriss requested a review from stevesloka November 30, 2020 18:23
@skriss
Copy link
Member Author

skriss commented Nov 30, 2020

The lint check is going to fail since this file has a non-conformant copyright header. Can add DCO when squashing for merge (the commit was created via GitHub GUI so doesn't have it).

@codecov
Copy link

codecov bot commented Nov 30, 2020

Codecov Report

Merging #3161 (55442d5) into main (e11d4ce) will not change coverage.
The diff coverage is 93.33%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3161   +/-   ##
=======================================
  Coverage   74.21%   74.21%           
=======================================
  Files          96       96           
  Lines        6011     6011           
=======================================
  Hits         4461     4461           
  Misses       1454     1454           
  Partials       96       96           
Impacted Files Coverage Δ
internal/k8s/helpers.go 91.66% <ø> (ø)
internal/status/proxystatus.go 80.95% <75.00%> (ø)
internal/contour/metrics.go 71.42% <100.00%> (ø)
internal/dag/extension_processor.go 93.93% <100.00%> (ø)
internal/fixture/detailedcondition.go 66.66% <100.00%> (ø)
internal/status/cache.go 88.88% <100.00%> (ø)
internal/status/extensionstatus.go 56.41% <100.00%> (ø)

Copy link
Member

@stevesloka stevesloka left a comment

Choose a reason for hiding this comment

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

/lgtm

@skriss skriss merged commit bccb50e into main Nov 30, 2020
@skriss skriss deleted the revert-3159-rm-kubeconditions branch January 21, 2021 16:51
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.

3 participants