Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Update README.md to cover attachments functionality#397

Merged
vdemeester merged 1 commit intodocker-archive-public:masterfrom
mikeparker:readme-attachments
Sep 27, 2018
Merged

Update README.md to cover attachments functionality#397
vdemeester merged 1 commit intodocker-archive-public:masterfrom
mikeparker:readme-attachments

Conversation

@mikeparker
Copy link
Copy Markdown
Contributor

@mikeparker mikeparker commented Sep 27, 2018

As #392 is now merged, I updated the README to include some description of the new attachments functionality.

image

Signed-off-by: Mike Parker <michael.parker@docker.com>
@mikeparker mikeparker changed the title Updates README.md to cover attachments functionality Update README.md to cover attachments functionality Sep 27, 2018
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 27, 2018

Codecov Report

Merging #397 into master will decrease coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #397      +/-   ##
==========================================
- Coverage   58.25%   58.12%   -0.13%     
==========================================
  Files          59       59              
  Lines        3260     3260              
==========================================
- Hits         1899     1895       -4     
- Misses       1095     1098       +3     
- Partials      266      267       +1
Impacted Files Coverage Δ
types/settings/settings.go 85.71% <0%> (-6.35%) ⬇️

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 6d16347...49b5695. Read the comment docs.

Copy link
Copy Markdown
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM

@vdemeester vdemeester merged commit 6717709 into docker-archive-public:master Sep 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants