Skip to content

Fix typo - #3247

Closed
milosa wants to merge 1 commit into
docker:masterfrom
milosa:patch-1
Closed

Fix typo#3247
milosa wants to merge 1 commit into
docker:masterfrom
milosa:patch-1

Conversation

@milosa

Copy link
Copy Markdown

No description provided.

@milosa
milosa requested a review from thaJeztah as a code ownerAugust 12, 2021 13:33
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #3247 (e59f622) into master (b4dde3a) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #3247 +/- ##
=======================================
Coverage 57.99% 57.99% =======================================
Files 302 302 Lines 21748 21748 =======================================
Hits 12613 12613 Misses 8212 8212 Partials 923 923 

@thaJeztahthaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! The change itself looks OK, but I see the commit is missing a DCO sign-off, which makes CI fail the failure shows some instructions on fixing that (unfortunately, it requires the commit to be amended on the CLI, and cannot be done through the GitHub web UI); https://github.com/docker/cli/pull/3247/checks?check_run_id=3312135659

When signing-off, make sure to use your real name (not the GitHub handle)
https://github.com/docker/cli/blob/master/CONTRIBUTING.md#sign-your-work

Let me know if you need help!

@thaJeztah

Copy link
Copy Markdown
Member

closing as this was addressed in 6fd4cff / #4712

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@milosa@codecov-commenter@thaJeztah