Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop v1util now that 0.3 is cut #884

Merged
merged 1 commit into from
Jan 4, 2021
Merged

Conversation

mattmoor
Copy link
Collaborator

Fixes: #873

@mattmoor
Copy link
Collaborator Author

mattmoor commented Dec 23, 2020

Most of those folks are VMware, so I'll post to a few internal slack channels.

EDIT: ugh... I'll do it in the new year. I don't have corp stuff on my M1 :P

@jromero
Copy link
Contributor

jromero commented Dec 23, 2020

Thanks @jonjohnsonjr.

Rerouting to @dfreilich :)

@codecov-io
Copy link

codecov-io commented Dec 23, 2020

Codecov Report

Merging #884 (7c86f1b) into master (c95d7bd) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #884   +/-   ##
=======================================
  Coverage   72.76%   72.76%           
=======================================
  Files         108      108           
  Lines        4655     4655           
=======================================
  Hits         3387     3387           
  Misses        758      758           
  Partials      510      510           
Impacted Files Coverage Δ
pkg/v1/tarball/layer.go 71.17% <100.00%> (ø)

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 c95d7bd...7c86f1b. Read the comment docs.

@dfreilich
Copy link

Thanks! We'll get notified automatically when dependabot picks up the new dependency, and it isn't successful, but we'll try to polish it off before then.

@mattmoor
Copy link
Collaborator Author

mattmoor commented Jan 4, 2021

Going to merge this, if folks are broken they can pin to the 0.3 release.

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.

Remove deprecated v1util
6 participants