[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1382 --- 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

[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-29 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1382#issuecomment-176653729 ran ```mvn clean --projects='org.apache.cloudstack:cloudstack' org.apache.rat:apache-rat-plugin:0.10:check``` with the following tail of output: ``` [I

[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-29 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1382#issuecomment-176651042 LGTM based on the lack of code --- 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

[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-28 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1382#issuecomment-176321558 LGTM based on the code. --- 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

[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-28 Thread ProjectMoon
GitHub user ProjectMoon opened a pull request: https://github.com/apache/cloudstack/pull/1382 Add missing license header to ActionEventUtilsTest. The test class was merged without the license header. This commit fixes that problem. Also note that the license header exists o

[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-28 Thread ProjectMoon
Github user ProjectMoon closed the pull request at: https://github.com/apache/cloudstack/pull/1381 --- 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 featu

[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-28 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/1381#issuecomment-176207310 My god I swear GitHub's PR interface can die in a fire. Wrong base branch again. --- If your project is set up for it, you can reply to this email and have you

[GitHub] cloudstack pull request: Add missing license header to ActionEvent...

2016-01-28 Thread ProjectMoon
GitHub user ProjectMoon opened a pull request: https://github.com/apache/cloudstack/pull/1381 Add missing license header to ActionEventUtilsTest. The test class was merged without the license header. This commit fixes that problem. Also note that the license header exists o