GUVNOR-2088 Adding logic to automatically publish to configured reposit...

[joewhite101]

GUVNOR-2088 Adding logic to automatically publish to configured repositories in the settings.xml so that the build and deploy from the WB matches mvn behavior

[manstis]

Thanks Joe.

67a1dc9

[psiroky]

Looking at the different length of indentation in the above code, it seems the code is usesing TABs. I guess the https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/blob/master/README.md does not strictly forbids it, but for the sake of consistency it would be better to use spaces.

[manstis]

Yes, you are correct.

The README does advise to configure both Eclipse and IDEA to use spaces
instead of tabs.

Fortunately I did not blindly merge the pull requests and hence the
committed code contains spaces.

On 24 March 2014 09:04, Petr Široký notifications@github.com wrote:

In
guvnor-m2repo-editor/guvnor-m2repo-editor-backend/src/main/java/org/guvnor/m2repo/backend/server/GuvnorM2Repository.java:

•                                           .addArtifact( jarArtifact )
  

•                                           .addArtifact( pomXMLArtifact )
  

•                                           .setRepository( remoteRepo );
  

•                                   Aether.getAether().getSystem().deploy( Aether.getAether().getSession(),settingsRequest );
  

•                               }
  

•                           }
  

•                       }
  

•                   }
  

•               }
  

•       }
  

•    }catch(DeploymentException de){
  

•       throw new RuntimeException( de );
  

•    }
  

  }

Looking at the different length of indentation in the above code, it seems
the code is usesing TABs. I guess the
https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/blob/master/README.mddoes not strictly forbids it, but for the sake of consistency it would be
better to use spaces.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/112/files#r10875968
.

[psiroky]

Argh, thanks for the info @manstis. Sorry, I should have looked at the actual commit that ended up in the repo.