Github user ahgittin commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/1119#issuecomment-166590578
awesome job. works great so far and should really simplify the migration,
as much as we can. great summary as well @johnmccabe.
i've been through all the PR's. here is my list of things to look at:
* poms:
* library repeats versions from software
* test deps were commented out in many places, look like they were
added but double check
* comment why `<relativePath/>` is used
* jsgui
* is it included correctly in Launcher
* comment BrooklynJavascriptGuiLauncher.java 71 / load order
* included correctly in release artifacts
* comment in root / brooklyn POM about order
* tests
* did java-web-app-simple.yaml vanish
* some tests are taking a long time
* overall
* double-check release artifacts
many are just sanity checks. i'll keep working through these.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
