projectcontour / contour-operator

Experimental repository to explore an operator for deploying Contour
Apache License 2.0
43 stars 34 forks source link

move Config type into internal/operator #485

Closed skriss closed 2 years ago

skriss commented 2 years ago

Drops the internal/config package and moves the Config struct into internal/operator where it's used.

Signed-off-by: Steve Kriss krisss@vmware.com

codecov[bot] commented 2 years ago

Codecov Report

Merging #485 (e12743b) into main (1296ee3) will decrease coverage by 3.11%. The diff coverage is 62.96%.

:exclamation: Current head e12743b differs from pull request most recent head c2bc51a. Consider uploading reports for the commit c2bc51a to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #485      +/-   ##
==========================================
- Coverage   80.76%   77.64%   -3.12%     
==========================================
  Files          29       29              
  Lines        2209     2210       +1     
==========================================
- Hits         1784     1716      -68     
- Misses        307      356      +49     
- Partials      118      138      +20     
Impacted Files Coverage Δ
cmd/contour-operator.go 0.00% <0.00%> (ø)
internal/controller/controller.go 72.13% <100.00%> (-8.20%) :arrow_down:
internal/objects/job/job.go 76.92% <100.00%> (-4.82%) :arrow_down:
internal/operator/config.go 100.00% <100.00%> (ø)
internal/operator/operator.go 77.77% <100.00%> (ø)
internal/status/status.go 48.64% <0.00%> (-24.33%) :arrow_down:
internal/objects/rbac.go 54.76% <0.00%> (-9.53%) :arrow_down:
internal/objects/configmap/configmap.go 70.88% <0.00%> (-6.33%) :arrow_down:
... and 4 more

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 1296ee3...c2bc51a. Read the comment docs.