If you're running simple compilation (build) on each patch, it'll be included. So no need for a special job. On 02/05/12 12:24, Eyal Edri wrote:
I believe we can add a checkstyle validation job in jenkins for that using checkstyle plugin[1].
[1] https://wiki.jenkins-ci.org/display/JENKINS/Checkstyle+Plugin
It can run on each commit, and soon on each patch sent to gerrit (once we'll boost our Jenkins framework for oVirt).
Eyal.
----- Original Message -----
From: "Alona Kaplan" <alkaplan@redhat.com> To: derez@redhat.com, gchaplik@redhat.com, achub@redhat.com, vszocs@redhat.com, tjelinek@redhat.com Cc: engine-devel@ovirt.org Sent: Wednesday, May 2, 2012 11:58:12 AM Subject: [Engine-devel] Fwd: Checkstyle- Webadmin & userportal localization
Hi,
Yesterday some changes were made in oVirt's checkstyle (http://gerrit.ovirt.org/#change,3760)
1. Renaming built-tools to build-tools-root and separating it into two packages: * checksyles- for checkstyle xml files. * ovirt-checkstyle-extension- for adding new (not built-in) checkstyle checks.
2. Adding to the checkstyle.xml a new check- NlsCheck, which fails the compilation in case non-externalized strings appear in the code.
[To understand more about the need of this check, see http://gerrit.ovirt.org/#change,3612]
From now on, all strings in the web-admin/user-portal java code should be one of the following: - Externalized (in case the string should be localized) - Have a "//$NON-NLS-N$" comment next to them (in case the string shouldn't be externalized, e.g. HashName of a component)
If there is a non externalized string in a web-admin/user-portal java file and there is no "//$NON-NLS-N$" comment next to it, mvn compilation will fail.
Note: NlsCheck is currently configured to run only on web-admin/user-portal projects. If you want NlsCheck check to run on another project, please add the following to the configuration of the checkstyle in the project's pom file: <propertyExpansion>runNlsCheck=true</propertyExpansion>
Alona.
-- /d "Common sense is not so common." --Voltaire, Dictionnaire Philosophique (1764)