[SQOOP-2425] Sqoop2: Improve our pre-commit hook - ASF JIRA

XML

Word

Printable

JSON

I would like to propose several improvements to our pre-commit hook, so that it's even more useful then it's now

Couple of ideas from top of my mind:

Order the actions that are performed. Right now the report always starts with ERRORS and I find it confusing. I would like to see order, something like "compilation success, unit test success, integration tests failed, ...).
Dependency checker. Detect if/when somebody adds a dependency and point it out (to verify that it's desired).
Save test log outputs, so that we can get to them. Their absence is making troubleshooting quite hard.
It would be great to add some additional static analysis - perhaps a checkstyle checker if we would have a rules to follow

I'm sure that there are other ides, so please don't hesitate and comment what you would like to see!

1.	Sqoop2: Precommit: Provide maven profile that will run all tests for precomit hook at once	Resolved	Jarek Jarcec Cecho
2.	Sqoop2: Precommit: Use the precommit maven profile to run tests	Resolved	Jarek Jarcec Cecho
3.	Sqoop2: Precommit: Print actions in order they've been executed	Resolved	Jarek Jarcec Cecho
4.	Sqoop2: Precommit: Add parameter --run-integration-tests	Resolved	Jarek Jarcec Cecho
5.	Sqoop2: Precommit: Add test coverage reports	Resolved	Jarek Jarcec Cecho
6.	Sqoop2: Precommit: Improve cobertura test coverage reporting	Resolved	Jarek Jarcec Cecho
7.	Sqoop2: Precommit: Introduce findbugs checks	Resolved	Jarek Jarcec Cecho
8.	Sqoop2: Precommit: Stop copying test results directly inside the pre-commit hook	Resolved	Jarek Jarcec Cecho
9.	Sqoop2: Precommit: Do not wipe out working directory on finish	Resolved	Jarek Jarcec Cecho
10.	"Modify" is misspelled in pre-commit hook	Resolved	Grant Henke