Skip to content

Fix .github/CODEOWNERS patterns - #356

Merged
vieux merged 1 commit into
docker:masterfrom
dnephin:fix-codeowners
Jul 19, 2017
Merged

Fix .github/CODEOWNERS patterns#356
vieux merged 1 commit into
docker:masterfrom
dnephin:fix-codeowners

Conversation

@dnephin

@dnephin dnephin commented Jul 18, 2017

Copy link
Copy Markdown
Contributor

Apparently they use https://git-scm.com/docs/gitignore#_pattern_format

I'm not sure if these ** are necessary, the doc suggests that just a / might work.

I tried testing these against my fork, but nothing works there. I also tried testing by modifying files in this PR, which also doesn't work. I'm guessing it has to be in master.

Signed-off-by: Daniel Nephin <dnephin@docker.com>
@codecov-io

codecov-io commented Jul 18, 2017

Copy link
Copy Markdown

Codecov Report

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

@@           Coverage Diff           @@
##           master     #356   +/-   ##
=======================================
  Coverage   45.47%   45.47%           
=======================================
  Files         193      193           
  Lines       16061    16061           
=======================================
  Hits         7304     7304           
  Misses       8379     8379           
  Partials      378      378

@vieux

vieux commented Jul 19, 2017

Copy link
Copy Markdown
Contributor

LGTM

@vieux
vieux merged commit 7f684c7 into docker:master Jul 19, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.07.0 milestone Jul 19, 2017
@dnephin
dnephin deleted the fix-codeowners branch July 19, 2017 00:19
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.

4 participants