All; This is just a heads up that we've added Checkstyle in both the 'foundation-2017' and 'develop' branches.
>From now on compilation will fail if the copyright headers are missing, or do >not match the expected template. For reference, the template regex is defined here: https://github.com/OpenNMS/opennms/blob/34b50cbccc62694056d91327249cf6b05ff55565/checkstyle/src/main/resources/AGPL-LICENSE.regexp.txt And the list of exceptions is maintained here: https://github.com/OpenNMS/opennms/blob/34b50cbccc62694056d91327249cf6b05ff55565/checkstyle/src/main/resources/license-suppressions.xml Aside from the license validation, I've added a simple check to forbid using 'com.google.common.base.Charsets' in favor of 'java.nio.charset.StandardCharsets': https://github.com/OpenNMS/opennms/blob/34b50cbccc62694056d91327249cf6b05ff55565/checkstyle/src/main/resources/nms_checks.xml I've opted to keep the number of checks to a minimum for now, but we can update/extend the list as need be. Best, Jesse ------------------------------------------------------------------------------ Announcing the Oxford Dictionaries API! The API offers world-renowned dictionary content that is easy and intuitive to access. Sign up for an account today to start using our lexical data to power your apps and projects. Get started today and enter our developer competition. http://sdm.link/oxford _______________________________________________ Please read the OpenNMS Mailing List FAQ: http://www.opennms.org/index.php/Mailing_List_FAQ opennms-devel mailing list To *unsubscribe* or change your subscription options, see the bottom of this page: https://lists.sourceforge.net/lists/listinfo/opennms-devel
