Added 'skip' options to goals.

[devang-gaur]

fixes #732 .

[devang-gaur]

Should I add some tests for this? @rhuss @rohanKanojia ?

[rohanKanojia]

It's fine without tests in my opinion

[codecov]

Codecov Report

Merging #1471 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #1471      +/-   ##
============================================
- Coverage     35.91%   35.91%   -0.01%     
  Complexity     1103     1103              
============================================
  Files           175      175              
  Lines          9914     9924      +10     
  Branches       1646     1647       +1     
============================================
+ Hits           3561     3564       +3     
- Misses         5939     5946       +7     
  Partials        414      414

[codecov]

Codecov Report

Merging #1471 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #1471      +/-   ##
============================================
- Coverage     35.91%   35.88%   -0.04%     
  Complexity     1103     1103              
============================================
  Files           175      175              
  Lines          9914     9924      +10     
  Branches       1646     1647       +1     
============================================
  Hits           3561     3561              
- Misses         5939     5949      +10     
  Partials        414      414

[Ladicek]

I just found by accident that many options are called docker.skip.* instead of fabric8.skip.*.

[rohanKanojia]

Our Push and Build goals inherit Push/Build goals from DMP. Do you think those options are being propagated here?

[Ladicek]

I don't know about that. I see that this pull request includes a bunch of parameters named docker.skip.*, even though the classes are in io.fabric8.maven.plugin.

[rohanKanojia]

ah, yeah. I can see it now 👓