Remove rules marked as deprecated in PMD (#12)

* Remove rules marked as deprecated in PMD

The following rules are marked as deprecated in PMD and will be removed in PMD 7.0:

* AbstractNaming
* ForLoopsMustUseBraces
* IfElseStmtsMustUseBraces
* IfStmtsMustUseBraces
* WhileLoopsMustUseBraces

Also, reformat CHANGELOG as org-mode format.
This commit is contained in:
Paul Campbell 2018-11-11 15:53:23 +00:00 committed by GitHub
parent d2fcdea310
commit 5cc35ed1fc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 51 additions and 48 deletions

View file

@ -1,39 +0,0 @@
CHANGELOG
=========
0.4.1
-----
* Bump kemitix-parent from 5.1.1 to 5.2.0 (#10)
0.4.0
-----
* Requires PMD 6.7.0
* The new Java rule FieldNamingConventions (java-codestyle)
* The new Java rule LinguisticNaming (java-codestyle)
* Remove deprecated rule VariableNamingConventions
* Remove deprecated rule SuspiciousConstantFieldName
0.3.0
-----
* Upgrade `kemitix-parent` to 5.1.1
* Remove unused tiles
* Disable PMD check `LawOfDemeter` due to Java Streams giving false positives
0.2.1
-----
* Upgrade `kemitix-parent` to 5.1.0
0.2.0
-----
* Disable `CommentSize` - didn't agree with MIT License headers
* Relaxed `AvoidDuplicateLiterals.maxDuplicateLiterals` - value is when to fail, not the max allowed
0.1.0
-----
* Initial release

51
CHANGELOG.org Normal file
View file

@ -0,0 +1,51 @@
* CHANGELOG
All notable changes to this project will be documented in this file.
The format is based on [Keep a
Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to
[Semantic Versioning](https://semver.org/spec/v2.0.0.html).
** 1.0.0 - 2018-11-11
*** Removed
The following rules are marked as deprecated in PMD and will be removed in
PMD 7.0:
* AbstractNaming
* ForLoopsMustUseBraces
* IfElseStmtsMustUseBraces
* IfStmtsMustUseBraces
* WhileLoopsMustUseBraces
*** Dependencies
* Bump kemitix-parent from 5.1.1 to 5.2.0
** 0.4.0
* Requires PMD 6.7.0
* The new Java rule FieldNamingConventions (java-codestyle)
* The new Java rule LinguisticNaming (java-codestyle)
* Remove deprecated rule VariableNamingConventions
* Remove deprecated rule SuspiciousConstantFieldName
** 0.3.0
* Upgrade `kemitix-parent` to 5.1.1
* Remove unused tiles
* Disable PMD check `LawOfDemeter` due to Java Streams giving false positives
** 0.2.1
* Upgrade `kemitix-parent` to 5.1.0
** 0.2.0
* Disable `CommentSize` - didn't agree with MIT License headers
* Relaxed `AvoidDuplicateLiterals.maxDuplicateLiterals` - value is when to fail, not the max allowed
** 0.1.0
* Initial release

View file

@ -69,11 +69,6 @@ http://pmd.sourceforge.net/ruleset/2.0.0 ">
<rule ref="category/java/bestpractices.xml/UseVarargs"/> <rule ref="category/java/bestpractices.xml/UseVarargs"/>
<!-- Code Style --> <!-- Code Style -->
<rule ref="category/java/codestyle.xml/AbstractNaming">
<properties>
<property name="strict" value="true"/>
</properties>
</rule>
<rule ref="category/java/codestyle.xml/AvoidDollarSigns"/> <rule ref="category/java/codestyle.xml/AvoidDollarSigns"/>
<rule ref="category/java/codestyle.xml/AvoidProtectedFieldInFinalClass"/> <rule ref="category/java/codestyle.xml/AvoidProtectedFieldInFinalClass"/>
<rule ref="category/java/codestyle.xml/AvoidProtectedMethodInFinalClassNotExtending"/> <rule ref="category/java/codestyle.xml/AvoidProtectedMethodInFinalClassNotExtending"/>
@ -100,10 +95,7 @@ http://pmd.sourceforge.net/ruleset/2.0.0 ">
</rule> </rule>
<rule ref="category/java/codestyle.xml/FieldNamingConventions" /> <rule ref="category/java/codestyle.xml/FieldNamingConventions" />
<rule ref="category/java/codestyle.xml/ForLoopShouldBeWhileLoop"/> <rule ref="category/java/codestyle.xml/ForLoopShouldBeWhileLoop"/>
<rule ref="category/java/codestyle.xml/ForLoopsMustUseBraces"/>
<rule ref="category/java/codestyle.xml/GenericsNaming"/> <rule ref="category/java/codestyle.xml/GenericsNaming"/>
<rule ref="category/java/codestyle.xml/IfElseStmtsMustUseBraces"/>
<rule ref="category/java/codestyle.xml/IfStmtsMustUseBraces"/>
<rule ref="category/java/codestyle.xml/LinguisticNaming" /> <rule ref="category/java/codestyle.xml/LinguisticNaming" />
<rule ref="category/java/codestyle.xml/LocalVariableCouldBeFinal"/> <rule ref="category/java/codestyle.xml/LocalVariableCouldBeFinal"/>
<rule ref="category/java/codestyle.xml/LongVariable"/> <rule ref="category/java/codestyle.xml/LongVariable"/>
@ -124,7 +116,6 @@ http://pmd.sourceforge.net/ruleset/2.0.0 ">
<rule ref="category/java/codestyle.xml/UnnecessaryReturn"/> <rule ref="category/java/codestyle.xml/UnnecessaryReturn"/>
<rule ref="category/java/codestyle.xml/UselessParentheses"/> <rule ref="category/java/codestyle.xml/UselessParentheses"/>
<rule ref="category/java/codestyle.xml/UselessQualifiedThis"/> <rule ref="category/java/codestyle.xml/UselessQualifiedThis"/>
<rule ref="category/java/codestyle.xml/WhileLoopsMustUseBraces"/>
<!-- Design --> <!-- Design -->
<rule ref="category/java/design.xml/AbstractClassWithoutAnyMethod"/> <rule ref="category/java/design.xml/AbstractClassWithoutAnyMethod"/>