[GitHub] brooklyn-dist issue #123: New license procedure

2018-07-17 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-dist/pull/123 Thanks @tbouron . Re YAML, yes, it should all be YAML, unobtrusively so, so it seems friendly for a person to look at, but also easy for consuming projects and people to analyse /

[GitHub] brooklyn-dist issue #123: New license procedure

2018-07-17 Thread tbouron
Github user tbouron commented on the issue: https://github.com/apache/brooklyn-dist/pull/123 One question though @ahgittin, is it expected that `NOTICE` and `LICENSE` file are now in a YAML format? ---

[GitHub] brooklyn-dist issue #123: New license procedure

2018-07-17 Thread tbouron
Github user tbouron commented on the issue: https://github.com/apache/brooklyn-dist/pull/123 @ahgittin LGTM, although it's very hard to do a throughout review with that amount of changes. I would like to have a second pair of eyes on this to verify it but I would be happy to merge.

[GitHub] brooklyn-dist issue #123: New license procedure

2018-06-26 Thread ahgittin
Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-dist/pull/123 Also note 6 other PRs -- one in each brooklyn project -- that are co-dependent on this. ---