Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

Use plugin.Name instead of string in all plugin configurations #362

Merged
merged 1 commit into from
Jan 14, 2017

Conversation

chungers
Copy link
Contributor

Standardize on the use of plugin.Name type for all plugin configs, instead of string.
This provides a common, standardized handling of plugin references and makes the code easier to read.

Signed-off-by: David Chung david.chung@docker.com

Signed-off-by: David Chung <david.chung@docker.com>
@chungers chungers changed the title Use plugin.Name instead of string in all plugin confiugrations Use plugin.Name instead of string in all plugin configurations Jan 14, 2017
@codecov-io
Copy link

Current coverage is 65.15% (diff: 72.22%)

Merging #362 into master will increase coverage by 10.74%

@@             master       #362   diff @@
==========================================
  Files            19         45     +26   
  Lines           816       2167   +1351   
  Methods           0          0           
  Messages          0          0           
  Branches          0          0           
==========================================
+ Hits            444       1412    +968   
- Misses          314        610    +296   
- Partials         58        145     +87   

Powered by Codecov. Last update 5a6c860...b09aadd

@chungers chungers merged commit a7d5dab into docker-archive:master Jan 14, 2017
@chungers chungers deleted the type-safe branch January 14, 2017 07:13
chungers pushed a commit to chungers/infrakit that referenced this pull request Sep 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants