Details
-
Improvement
-
Status: Open
-
Major
-
Resolution: Unresolved
-
None
-
None
-
None
-
None
Description
Currently, checkstyle rules are configured as warning and the pre commit build reports issues like line ending with space, line longer than 132 chars etc. Instead, if checkstyle rules are configured as errors, the local build itself will fail marking the exact line causing violation. This is much easier to fix.
To enable checkstyle rules strictly, we need to fix the existing violations. So, I am going to submit 2 patches:
1. Whole oozie codebase reformatting using idea. This reduces the violations to small number which can be later fixed manually. This will be a huge patch, but doesn't contain any manual changes
2. Fix the remaining checkstyle violations
Attachments
1.
|
Fix Headers in java code |
|
Resolved | Shwetha GS |
2.
|
Fix checkstyle issues |
|
Open | Unassigned |
3.
|
Building modules independently |
|
Resolved | Shwetha GS |
4.
|
Display patch analysis issues |
|
Open | Unassigned |
5.
|
Fix Checkstyle issues in oozie-tools |
|
Resolved | Jan Hentschel |
6.
|
Fix Checkstyle issues in oozie-client |
|
Resolved | Jan Hentschel |