Github user tbouron commented on the issue:

    https://github.com/apache/brooklyn-library/pull/160
  
    This looks fine @aledsage but I can see 2 issues there:
    1. what about the `settings.xml`? In the PR context, we don't care because 
we don't push artifacts to maven central but as soon as it is merged, the real 
jenkins job will try that and will then fail because the credentials are stored 
there
    2. This means that we will need to add the same `<exclude>...</exclude>` 
lines to every brooklyn submodule


---

Reply via email to