Add RootDir property to GraphData#585
Conversation
Adds RootDir property to GraphData. This was seen on a CentOS system without aufs support.
|
Ooops, sorry. Looks like I forgot to turn off my team's code formatter. Let me swap and resubmit |
Adds RootDir property to GraphData. This was seen on a CentOS system without aufs support.
| import org.apache.commons.lang.builder.ToStringBuilder; | ||
|
|
||
| import javax.annotation.CheckForNull; | ||
| import com.fasterxml.jackson.annotation.JsonProperty; |
There was a problem hiding this comment.
yeah sorry, that was my other code formatter.
|
Please remove all coding style changes. |
|
IDEA default or eclipse would mostly match, for eclipse some config file even committed in repository. For idea i still had no time, sorry for it. Formatting really makes difficult reviews. |
|
@KostyaSha yeah sorry, going to reformat and resubmit. I found the code style under the /etc directory. Reformatting with that. |
Adds RootDir property to GraphData. This was seen on a CentOS system without aufs support.
Current coverage is 23.18%
@@ master #585 diff @@
==========================================
Files 296 296
Lines 6157 6160 +3
Methods 0 0
Messages 0 0
Branches 539 539
==========================================
+ Hits 1423 1428 +5
+ Misses 4642 4640 -2
Partials 92 92
|
| return rootDir; | ||
| } | ||
|
|
||
| /** |
There was a problem hiding this comment.
Please fix comment. Thanks.
There was a problem hiding this comment.
Oops, just amended to fix that. Sorry!
|
Now looks good :) Please ignore commit status, it was PR builder against docker-beta that i started configuring :) |
Adds RootDir property to GraphData. This was seen on a CentOS system without aufs support.
4 participants
Adds RootDir property to GraphData.
This was seen on a CentOS system without aufs support.
This change is