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

Fix grammar in docs by adding omitted word #4121

Merged
merged 1 commit into from
Mar 30, 2023

Conversation

developerjake
Copy link
Contributor

- What I did
Edited second paragraph under ### Daemon configuration file to change "regardless their value" to "regardless of their value"
- How I did it
Edited second paragraph under ### Daemon configuration file to change "regardless their value" to "regardless of their value"
- How to verify it
Look
- Description for the changelog

Edited second paragraph under ### Daemon configuration file to change "regardless their value" to "regardless of their value"

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

@codecov-commenter
Copy link

Codecov Report

Merging #4121 (13864a2) into master (f5d698a) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4121      +/-   ##
==========================================
- Coverage   59.16%   59.15%   -0.01%     
==========================================
  Files         287      285       -2     
  Lines       24716    24713       -3     
==========================================
- Hits        14623    14619       -4     
  Misses       9209     9209              
- Partials      884      885       +1     

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@thaJeztah
Copy link
Member

thaJeztah commented Mar 26, 2023

CI failures are unrelated; Looks like some fixtures need updating;

Error: error contacting notary server: x509: certificate has expired or is not yet valid: current time 2023-03-26T12:52:16Z is after 2023-03-25T14:47:57Z
=== FAIL: e2e/container TestCreateWithContentTrust (0.24s)
    create_test.go:18: assertion failed: 
        Command:  docker image push registry:5000/trust-create:latest
        ExitCode: 1
        Error:    exit status 1
        Stdout:   The push refers to repository [registry:5000/trust-create]
        5bef08742407: Preparing
        5bef08742407: Mounted from alpine
        latest: digest: sha256:641b95ddb2ea9dc2af1a0113b6b348ebc20872ba615204fbe12148e98fd6f23d size: 528
        Signing and pushing trust metadata
        
        Stderr:   Error: error contacting notary server: x509: certificate has expired or is not yet valid: current time 2023-03-26T12:52:08Z is after 2023-03-25T14:47:57Z
        
        
        Failures:
        ExitCode was 1 expected 0
        Expected no error

Edited second paragraph under ### Daemon configuration file to change "regardless their value" to "regardless of their value"

Signed-off-by: Jake Stokes <[email protected]>
@thaJeztah
Copy link
Member

did a quick rebase, as CI should be fixed now 👍

@thaJeztah
Copy link
Member

All green now 🎉

Sign up for free to 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