Skip to content

Reference common project repo contents#2716

Merged
crosbymichael merged 2 commits intocontainerd:masterfrom
estesp:reference-common-project-repo
Oct 15, 2018
Merged

Reference common project repo contents#2716
crosbymichael merged 2 commits intocontainerd:masterfrom
estesp:reference-common-project-repo

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Oct 15, 2018

Now that we have all the common scripting and project documents in containerd/project, remove local copies of these files and use/reference the common project repo as the source location.

Now that all the content is set up appropriately in the
containerd/project repo, remove local copies and point to the common
project content from the containerd README.md

Signed-off-by: Phil Estes <[email protected]>
Remove local copies of common containerd/project located scripts for
DCO, fileheader, and vendor checks.

Signed-off-by: Phil Estes <[email protected]>
@estesp estesp force-pushed the reference-common-project-repo branch from 58b4d5b to bd93a66 Compare October 15, 2018 13:05
@codecov-io
Copy link
Copy Markdown

codecov-io commented Oct 15, 2018

Codecov Report

Merging #2716 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2716   +/-   ##
=======================================
  Coverage   43.54%   43.54%           
=======================================
  Files         101      101           
  Lines       10734    10734           
=======================================
  Hits         4674     4674           
  Misses       5326     5326           
  Partials      734      734
Flag Coverage Δ
#linux 47.22% <ø> (ø) ⬆️
#windows 40.67% <ø> (ø) ⬆️

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 0146a48...bd93a66. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants