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

opts: remove unused RuntimeOpt #3769

Merged
merged 1 commit into from
Sep 6, 2022
Merged

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Sep 3, 2022

This type was copied from the daemon in 8b725e1 (#82), but actually unused on the CLI side, so we can remove it here.

The original code is still present in the daemon at
https://github.com/moby/moby/blob/7b9275c0da707b030e62c96b679a976f31f929d3/opts/runtime.go

- A picture of a cute animal (not mandatory but encouraged)

This type was copied from the daemone in 8b725e1,
but actually unused on the CLI side, so we can remove it here.

The original code is still present in the daemon at
https://github.com/moby/moby/blob/7b9275c0da707b030e62c96b679a976f31f929d3/opts/runtime.go

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Sep 3, 2022
@thaJeztah thaJeztah added this to the 22.06.0 milestone Sep 3, 2022
@codecov-commenter
Copy link

codecov-commenter commented Sep 3, 2022

Codecov Report

Merging #3769 (c645c99) into master (52b89d7) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3769      +/-   ##
==========================================
+ Coverage   59.14%   59.23%   +0.09%     
==========================================
  Files         289      288       -1     
  Lines       24681    24641      -40     
==========================================
  Hits        14597    14597              
+ Misses       9213     9173      -40     
  Partials      871      871              

@thaJeztah thaJeztah merged commit 7577435 into docker:master Sep 6, 2022
@thaJeztah thaJeztah deleted the opts_deadcode branch September 6, 2022 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/refactor PR's that refactor, or clean-up code status/2-code-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants