Skip to content

cli/command: add Cli.CurrentVersion() function - #3885

Merged
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:context_lazy_evaluate_step2
Nov 28, 2022
Merged

cli/command: add Cli.CurrentVersion() function#3885
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:context_lazy_evaluate_step2

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

This internalizes constructing the Client(), which allows us to provide fallbacks when trying to determin the current API version.

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

This internalizes constructing the Client(), which allows us to provide
fallbacks when trying to determin the current API version.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztahthaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Nov 28, 2022
@thaJeztahthaJeztah added this to the 23.0.0 milestone Nov 28, 2022
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #3885 (a7e2c3e) into master (cbf0522) will decrease coverage by 0.00%.
The diff coverage is 7.14%.

Additional details and impacted files
@@ Coverage Diff @@## master #3885 +/- ##
==========================================
- Coverage 59.22% 59.22% -0.01% 
==========================================
Files 285 287 +2 Lines 24653 24661 +8 ==========================================
+ Hits 14601 14605 +4 - Misses 9168 9173 +5 + Partials 884 883 -1 

@silvin-lubeckisilvin-lubecki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@thaJeztah

Copy link
Copy Markdown
MemberAuthor

Thanks! Let me bring this one in 👍

@thaJeztah
thaJeztah merged commit 56580e7 into docker:masterNov 28, 2022
@thaJeztah
thaJeztah deleted the context_lazy_evaluate_step2 branch November 28, 2022 10:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactorPR's that refactor, or clean-up codestatus/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@thaJeztah@codecov-commenter@silvin-lubecki