Example of violations:
android {
diff --git a/android-plugin/src/test/java/io/ecocode/java/JavaRulesDefinitionTest.java b/android-plugin/src/test/java/io/ecocode/java/JavaRulesDefinitionTest.java
index 91485991..bb099ddc 100644
--- a/android-plugin/src/test/java/io/ecocode/java/JavaRulesDefinitionTest.java
+++ b/android-plugin/src/test/java/io/ecocode/java/JavaRulesDefinitionTest.java
@@ -47,7 +47,7 @@ public void test() {
private void assertSocialRuleProperties(Repository repository) {
- Rule googleTrackerRule = repository.rule("EC532");
+ Rule googleTrackerRule = repository.rule("GCI532");
assertThat(googleTrackerRule).isNotNull();
assertThat(googleTrackerRule.name()).isEqualTo("Privacy: Google Tracker");
assertThat(googleTrackerRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
@@ -56,193 +56,193 @@ private void assertSocialRuleProperties(Repository repository) {
private void assertEnergyRuleProperties(Repository repository) {
- Rule sensorCoalesceRule = repository.rule("EC500");
+ Rule sensorCoalesceRule = repository.rule("GCI500");
assertThat(sensorCoalesceRule).isNotNull();
assertThat(sensorCoalesceRule.name()).isEqualTo("Batch: Sensor Coalesce");
assertThat(sensorCoalesceRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(sensorCoalesceRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule jobCoalesceRule = repository.rule("EC501");
+ Rule jobCoalesceRule = repository.rule("GCI501");
assertThat(jobCoalesceRule).isNotNull();
assertThat(jobCoalesceRule.name()).isEqualTo("Batch: Job Coalesce");
assertThat(jobCoalesceRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(jobCoalesceRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule internetInTheLoopRule = repository.rule("EC502");
+ Rule internetInTheLoopRule = repository.rule("GCI502");
assertThat(internetInTheLoopRule).isNotNull();
assertThat(internetInTheLoopRule.name()).isEqualTo("Bottleneck: Internet In The Loop");
assertThat(internetInTheLoopRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(internetInTheLoopRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule wifiMulticastLockRule = repository.rule("EC503");
+ Rule wifiMulticastLockRule = repository.rule("GCI503");
assertThat(wifiMulticastLockRule).isNotNull();
assertThat(wifiMulticastLockRule.name()).isEqualTo("Bottleneck: Wifi Multicast Lock");
assertThat(wifiMulticastLockRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(wifiMulticastLockRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule uncompressedDataTransmissionRule = repository.rule("EC504");
+ Rule uncompressedDataTransmissionRule = repository.rule("GCI504");
assertThat(uncompressedDataTransmissionRule).isNotNull();
assertThat(uncompressedDataTransmissionRule.name()).isEqualTo("Bottleneck: Uncompressed Data Transmission");
assertThat(uncompressedDataTransmissionRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(uncompressedDataTransmissionRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule keepScreenOnAddFlagsRule = repository.rule("EC505");
+ Rule keepScreenOnAddFlagsRule = repository.rule("GCI505");
assertThat(keepScreenOnAddFlagsRule).isNotNull();
assertThat(keepScreenOnAddFlagsRule.name()).isEqualTo("Idleness: Keep Screen On (addFlags)");
assertThat(keepScreenOnAddFlagsRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(keepScreenOnAddFlagsRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule keepScreenOnSetFlagsRule = repository.rule("EC506");
+ Rule keepScreenOnSetFlagsRule = repository.rule("GCI506");
assertThat(keepScreenOnSetFlagsRule).isNotNull();
assertThat(keepScreenOnSetFlagsRule.name()).isEqualTo("Idleness: Keep Screen On (setFlags)");
assertThat(keepScreenOnSetFlagsRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(keepScreenOnSetFlagsRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule keepCpuOnRule = repository.rule("EC507");
+ Rule keepCpuOnRule = repository.rule("GCI507");
assertThat(keepCpuOnRule).isNotNull();
assertThat(keepCpuOnRule.name()).isEqualTo("Idleness: Keep Cpu On");
assertThat(keepCpuOnRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(keepCpuOnRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule durableWakeLockRule = repository.rule("EC508");
+ Rule durableWakeLockRule = repository.rule("GCI508");
assertThat(durableWakeLockRule).isNotNull();
assertThat(durableWakeLockRule.name()).isEqualTo("Idleness: Durable Wake Lock");
assertThat(durableWakeLockRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(durableWakeLockRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule rigidAlarmRule = repository.rule("EC509");
+ Rule rigidAlarmRule = repository.rule("GCI509");
assertThat(rigidAlarmRule).isNotNull();
assertThat(rigidAlarmRule.name()).isEqualTo("Idleness: Rigid Alarm");
assertThat(rigidAlarmRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(rigidAlarmRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule continuousRenderingRule = repository.rule("EC510");
+ Rule continuousRenderingRule = repository.rule("GCI510");
assertThat(continuousRenderingRule).isNotNull();
assertThat(continuousRenderingRule.name()).isEqualTo("Idleness: Continuous Rendering");
assertThat(continuousRenderingRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(continuousRenderingRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule keepVoiceAwakeRule = repository.rule("EC511");
+ Rule keepVoiceAwakeRule = repository.rule("GCI511");
assertThat(keepVoiceAwakeRule).isNotNull();
assertThat(keepVoiceAwakeRule.name()).isEqualTo("Idleness: Keep Voice Awake");
assertThat(keepVoiceAwakeRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(keepVoiceAwakeRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule cameraLeakRule = repository.rule("EC512");
+ Rule cameraLeakRule = repository.rule("GCI512");
assertThat(cameraLeakRule).isNotNull();
assertThat(cameraLeakRule.name()).isEqualTo("Leakage: Camera Leak");
assertThat(cameraLeakRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(cameraLeakRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule locationLeakRule = repository.rule("EC513");
+ Rule locationLeakRule = repository.rule("GCI513");
assertThat(locationLeakRule).isNotNull();
assertThat(locationLeakRule.name()).isEqualTo("Leakage: Location Leak");
assertThat(locationLeakRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(locationLeakRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule sensorManagerLeakRule = repository.rule("EC514");
+ Rule sensorManagerLeakRule = repository.rule("GCI514");
assertThat(sensorManagerLeakRule).isNotNull();
assertThat(sensorManagerLeakRule.name()).isEqualTo("Leakage: SensorManager Leak");
assertThat(sensorManagerLeakRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(sensorManagerLeakRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule mediaLeakMediaRecorderLeakRule = repository.rule("EC515");
+ Rule mediaLeakMediaRecorderLeakRule = repository.rule("GCI515");
assertThat(mediaLeakMediaRecorderLeakRule).isNotNull();
assertThat(mediaLeakMediaRecorderLeakRule.name()).isEqualTo("Leakage: Media Leak (MediaRecorder)");
assertThat(mediaLeakMediaRecorderLeakRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(mediaLeakMediaRecorderLeakRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule mediaLeakMediaPlayerLeakRule = repository.rule("EC516");
+ Rule mediaLeakMediaPlayerLeakRule = repository.rule("GCI516");
assertThat(mediaLeakMediaPlayerLeakRule).isNotNull();
assertThat(mediaLeakMediaPlayerLeakRule.name()).isEqualTo("Leakage: Media Leak (MediaPlayer)");
assertThat(mediaLeakMediaPlayerLeakRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(mediaLeakMediaPlayerLeakRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule fusedLocationRule = repository.rule("EC517");
+ Rule fusedLocationRule = repository.rule("GCI517");
assertThat(fusedLocationRule).isNotNull();
assertThat(fusedLocationRule.name()).isEqualTo("Optimized API: Fused Location");
assertThat(fusedLocationRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(fusedLocationRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule bluetoothLowEnergyRule = repository.rule("EC518");
+ Rule bluetoothLowEnergyRule = repository.rule("GCI518");
assertThat(bluetoothLowEnergyRule).isNotNull();
assertThat(bluetoothLowEnergyRule.name()).isEqualTo("Optimized API: Bluetooth Low-Energy");
assertThat(bluetoothLowEnergyRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(bluetoothLowEnergyRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule chargeAwarenessRule = repository.rule("EC519");
+ Rule chargeAwarenessRule = repository.rule("GCI519");
assertThat(chargeAwarenessRule).isNotNull();
assertThat(chargeAwarenessRule.name()).isEqualTo("Power: Charge Awareness");
assertThat(chargeAwarenessRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(chargeAwarenessRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule saveModeAwareness = repository.rule("EC520");
+ Rule saveModeAwareness = repository.rule("GCI520");
assertThat(saveModeAwareness).isNotNull();
assertThat(saveModeAwareness.name()).isEqualTo("Power: Save Mode Awareness");
assertThat(saveModeAwareness.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(saveModeAwareness.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule thriftyMotionSensorRule = repository.rule("EC521");
+ Rule thriftyMotionSensorRule = repository.rule("GCI521");
assertThat(thriftyMotionSensorRule).isNotNull();
assertThat(thriftyMotionSensorRule.name()).isEqualTo("Sobriety: Thrifty Motion Sensor");
assertThat(thriftyMotionSensorRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(thriftyMotionSensorRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule brightnessOverrideRule = repository.rule("EC522");
+ Rule brightnessOverrideRule = repository.rule("GCI522");
assertThat(brightnessOverrideRule).isNotNull();
assertThat(brightnessOverrideRule.name()).isEqualTo("Sobriety: Brightness Override");
assertThat(brightnessOverrideRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(brightnessOverrideRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule thriftyGeolocationMinTimeRule = repository.rule("EC523");
+ Rule thriftyGeolocationMinTimeRule = repository.rule("GCI523");
assertThat(thriftyGeolocationMinTimeRule).isNotNull();
assertThat(thriftyGeolocationMinTimeRule.name()).isEqualTo("Sobriety: Thrifty Geolocation (minTime)");
assertThat(thriftyGeolocationMinTimeRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(thriftyGeolocationMinTimeRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule thriftyGeolocationCriteriaRule = repository.rule("EC524");
+ Rule thriftyGeolocationCriteriaRule = repository.rule("GCI524");
assertThat(thriftyGeolocationCriteriaRule).isNotNull();
assertThat(thriftyGeolocationCriteriaRule.name()).isEqualTo("Sobriety: Thrifty Geolocation Criteria");
assertThat(thriftyGeolocationCriteriaRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(thriftyGeolocationCriteriaRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule thriftyBluetoothLowEnergySetAdvertiseMode = repository.rule("EC525");
+ Rule thriftyBluetoothLowEnergySetAdvertiseMode = repository.rule("GCI525");
assertThat(thriftyBluetoothLowEnergySetAdvertiseMode).isNotNull();
assertThat(thriftyBluetoothLowEnergySetAdvertiseMode.name()).isEqualTo("Sobriety: Thrifty Bluetooth Low Energy (setAdvertiseMode)");
assertThat(thriftyBluetoothLowEnergySetAdvertiseMode.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(thriftyBluetoothLowEnergySetAdvertiseMode.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule thriftyBluetoothLowEnergyRequestConnectionPriority = repository.rule("EC526");
+ Rule thriftyBluetoothLowEnergyRequestConnectionPriority = repository.rule("GCI526");
assertThat(thriftyBluetoothLowEnergyRequestConnectionPriority).isNotNull();
assertThat(thriftyBluetoothLowEnergyRequestConnectionPriority.name()).isEqualTo("Sobriety: Thrifty Bluetooth Low Energy (requestConnectionPriority)");
assertThat(thriftyBluetoothLowEnergyRequestConnectionPriority.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(thriftyBluetoothLowEnergyRequestConnectionPriority.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule thriftyGeolocationMinDistanceRule = repository.rule("EC527");
+ Rule thriftyGeolocationMinDistanceRule = repository.rule("GCI527");
assertThat(thriftyGeolocationMinDistanceRule).isNotNull();
assertThat(thriftyGeolocationMinDistanceRule.name()).isEqualTo("Sobriety: Thrifty Geolocation (minDistance)");
assertThat(thriftyGeolocationMinDistanceRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(thriftyGeolocationMinDistanceRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule vibrationFreeRule = repository.rule("EC528");
+ Rule vibrationFreeRule = repository.rule("GCI528");
assertThat(vibrationFreeRule).isNotNull();
assertThat(vibrationFreeRule.name()).isEqualTo("Sobriety: Vibration Free");
assertThat(vibrationFreeRule.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(vibrationFreeRule.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule thriftyNotification = repository.rule("EC529");
+ Rule thriftyNotification = repository.rule("GCI529");
assertThat(thriftyNotification).isNotNull();
assertThat(thriftyNotification.name()).isEqualTo("Sobriety: Thrifty Notification");
assertThat(thriftyNotification.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(thriftyNotification.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule torchFree = repository.rule("EC530");
+ Rule torchFree = repository.rule("GCI530");
assertThat(torchFree).isNotNull();
assertThat(torchFree.name()).isEqualTo("Sobriety: Torch Free");
assertThat(torchFree.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
assertThat(torchFree.type()).isEqualTo(RuleType.CODE_SMELL);
- Rule highFrameRate = repository.rule("EC531");
+ Rule highFrameRate = repository.rule("GCI531");
assertThat(highFrameRate).isNotNull();
assertThat(highFrameRate.name()).isEqualTo("Sobriety: High Frame Rate");
assertThat(highFrameRate.debtRemediationFunction().type()).isEqualTo(Type.CONSTANT_ISSUE);
diff --git a/android-plugin/src/test/java/io/ecocode/xml/XmlRulesDefinitionTest.java b/android-plugin/src/test/java/io/ecocode/xml/XmlRulesDefinitionTest.java
index f3738826..0d11ae7f 100644
--- a/android-plugin/src/test/java/io/ecocode/xml/XmlRulesDefinitionTest.java
+++ b/android-plugin/src/test/java/io/ecocode/xml/XmlRulesDefinitionTest.java
@@ -35,39 +35,39 @@ public void test() {
assertThat(repository.language()).isEqualTo(Xml.KEY);
assertThat(repository.rules()).hasSize(XmlCheckList.getXmlChecks().size());
- RulesDefinition.Rule serviceBootTimeXmlRule = repository.rule("EC540");
+ RulesDefinition.Rule serviceBootTimeXmlRule = repository.rule("GCI540");
assertThat(serviceBootTimeXmlRule).isNotNull();
assertThat(serviceBootTimeXmlRule.name()).isEqualTo("Batch: Service Boot Time");
- RulesDefinition.Rule keepScreenOnXmlRule = repository.rule("EC541");
+ RulesDefinition.Rule keepScreenOnXmlRule = repository.rule("GCI541");
assertThat(keepScreenOnXmlRule).isNotNull();
assertThat(keepScreenOnXmlRule.name()).isEqualTo("Idleness: Keep Screen On");
- RulesDefinition.Rule keepCpuOnXmlRule = repository.rule("EC542");
+ RulesDefinition.Rule keepCpuOnXmlRule = repository.rule("GCI542");
assertThat(keepCpuOnXmlRule).isNotNull();
assertThat(keepCpuOnXmlRule.name()).isEqualTo("Idleness: Keep CPU On");
- RulesDefinition.Rule compagnionInBackgroundXmlRule = repository.rule("EC543");
+ RulesDefinition.Rule compagnionInBackgroundXmlRule = repository.rule("GCI543");
assertThat(compagnionInBackgroundXmlRule).isNotNull();
assertThat(compagnionInBackgroundXmlRule.name()).isEqualTo("Power: Compagnion In Background");
- RulesDefinition.Rule ignoreBatteryOptimizationsXmlRule = repository.rule("EC544");
+ RulesDefinition.Rule ignoreBatteryOptimizationsXmlRule = repository.rule("GCI544");
assertThat(ignoreBatteryOptimizationsXmlRule).isNotNull();
assertThat(ignoreBatteryOptimizationsXmlRule.name()).isEqualTo("Power: Ignore Battery Optimizations");
- RulesDefinition.Rule chargeAwarenessXmlRule = repository.rule("EC545");
+ RulesDefinition.Rule chargeAwarenessXmlRule = repository.rule("GCI545");
assertThat(chargeAwarenessXmlRule).isNotNull();
assertThat(chargeAwarenessXmlRule.name()).isEqualTo("Power: Charge Awareness");
- RulesDefinition.Rule DarkUIBrightColorsXmlRule = repository.rule("EC547");
+ RulesDefinition.Rule DarkUIBrightColorsXmlRule = repository.rule("GCI547");
assertThat(DarkUIBrightColorsXmlRule).isNotNull();
assertThat(DarkUIBrightColorsXmlRule.name()).isEqualTo("Sobriety: Dark UI (Bright Colors)");
- RulesDefinition.Rule DarkUIThemeXmlRule = repository.rule("EC548");
+ RulesDefinition.Rule DarkUIThemeXmlRule = repository.rule("GCI548");
assertThat(DarkUIThemeXmlRule).isNotNull();
assertThat(DarkUIThemeXmlRule.name()).isEqualTo("Sobriety: Dark UI (Theme)");
- RulesDefinition.Rule saveModeAwarenessXml = repository.rule("EC546");
+ RulesDefinition.Rule saveModeAwarenessXml = repository.rule("GCI546");
assertThat(saveModeAwarenessXml).isNotNull();
assertThat(saveModeAwarenessXml.name()).isEqualTo("Power: Save Mode Awareness");
diff --git a/codenarc-converter/CodeNarc/build.gradle b/codenarc-converter/CodeNarc/build.gradle
index 50e3b8df..eab46dec 100644
--- a/codenarc-converter/CodeNarc/build.gradle
+++ b/codenarc-converter/CodeNarc/build.gradle
@@ -14,7 +14,7 @@ shadowJar {
group = 'org.codenarc'
archivesBaseName = 'CodeNarc'
-version = '2.2.5'
+version = '3.5.0'
sourceCompatibility = '1.7'
targetCompatibility = '1.7'
diff --git a/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRules.groovy.txt b/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRules.groovy.txt
index 85979380..6a7c9117 100644
--- a/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRules.groovy.txt
+++ b/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRules.groovy.txt
@@ -87,7 +87,7 @@ ruleset {
DuplicateNumberLiteral
DuplicateSetValue
DuplicateStringLiteral
- EC5001
+ GCI5001
ElseBlockBraces
EmptyCatchBlock
EmptyClass
@@ -402,4 +402,4 @@ ruleset {
WaitOutsideOfWhileLoop
WhileStatementBraces
-}
\ No newline at end of file
+}
diff --git a/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRulesByCategory.groovy.txt b/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRulesByCategory.groovy.txt
index 35488472..003b04ae 100644
--- a/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRulesByCategory.groovy.txt
+++ b/codenarc-converter/CodeNarc/docs/StarterRuleSet-AllRulesByCategory.groovy.txt
@@ -166,8 +166,8 @@ ruleset {
DuplicateStringLiteral
// rulesets/ecocode.xml
- EC5001
- EC5003
+ GCI5001
+ GCI5003
// rulesets/enhanced.xml
CloneWithoutCloneable
@@ -451,4 +451,4 @@ ruleset {
UnusedVariable
-}
\ No newline at end of file
+}
diff --git a/codenarc-converter/CodeNarc/docs/codenarc-rule-index-by-name.md b/codenarc-converter/CodeNarc/docs/codenarc-rule-index-by-name.md
index 2c117951..c05b0333 100644
--- a/codenarc-converter/CodeNarc/docs/codenarc-rule-index-by-name.md
+++ b/codenarc-converter/CodeNarc/docs/codenarc-rule-index-by-name.md
@@ -89,7 +89,7 @@ title: CodeNarc - Rule Index by Name
* [DuplicateNumberLiteral](./codenarc-rules-dry.html#duplicatenumberliteral-rule)
* [DuplicateSetValue](./codenarc-rules-basic.html#duplicatesetvalue-rule)
* [DuplicateStringLiteral](./codenarc-rules-dry.html#duplicatestringliteral-rule)
-* [EC5001](./codenarc-rules-ecocode.html#EC5001-rule)
+* [GCI5001](./codenarc-rules-ecocode.html#GCI5001-rule)
* [ElseBlockBraces](./codenarc-rules-braces.html#elseblockbraces-rule)
* [EmptyCatchBlock](./codenarc-rules-basic.html#emptycatchblock-rule)
* [EmptyClass](./codenarc-rules-basic.html#emptyclass-rule)
diff --git a/codenarc-converter/CodeNarc/docs/codenarc-rule-index.md b/codenarc-converter/CodeNarc/docs/codenarc-rule-index.md
index 616375b1..f9e532a5 100644
--- a/codenarc-converter/CodeNarc/docs/codenarc-rule-index.md
+++ b/codenarc-converter/CodeNarc/docs/codenarc-rule-index.md
@@ -167,7 +167,7 @@ title: CodeNarc - Rule Index
* [DuplicateStringLiteral](./codenarc-rules-dry.html#duplicatestringliteral-rule)
## [Ecocode](./codenarc-rules-ecocode.html)
-* [EC5001](./codenarc-rules-ecocode.html#EC5001-rule)
+* [GCI5001](./codenarc-rules-ecocode.html#GCI5001-rule)
## [Enhanced](./codenarc-rules-enhanced.html)
* [CloneWithoutCloneable](./codenarc-rules-enhanced.html#clonewithoutcloneable-rule)
diff --git a/codenarc-converter/CodeNarc/docs/codenarc-rules-ecocode.md b/codenarc-converter/CodeNarc/docs/codenarc-rules-ecocode.md
index 6280f050..e31dd1cc 100644
--- a/codenarc-converter/CodeNarc/docs/codenarc-rules-ecocode.md
+++ b/codenarc-converter/CodeNarc/docs/codenarc-rules-ecocode.md
@@ -5,7 +5,7 @@ title: CodeNarc - Ecocode Rules
# Ecocode Rules ("*rulesets/ecocode.xml*")
-## EC5001 Rule
+## GCI5001 Rule
*Since CodeNarc 2.2.1*
@@ -45,7 +45,7 @@ Example of violations:
}
```
-## EC5003 Rule
+## GCI5003 Rule
*Since CodeNarc 2.2.3*
diff --git a/codenarc-converter/CodeNarc/src/main/groovy/org/codenarc/rule/ecocode/EC5001.groovy b/codenarc-converter/CodeNarc/src/main/groovy/org/codenarc/rule/ecocode/GCI5001.groovy
similarity index 92%
rename from codenarc-converter/CodeNarc/src/main/groovy/org/codenarc/rule/ecocode/EC5001.groovy
rename to codenarc-converter/CodeNarc/src/main/groovy/org/codenarc/rule/ecocode/GCI5001.groovy
index e91f40df..c1738655 100644
--- a/codenarc-converter/CodeNarc/src/main/groovy/org/codenarc/rule/ecocode/EC5001.groovy
+++ b/codenarc-converter/CodeNarc/src/main/groovy/org/codenarc/rule/ecocode/GCI5001.groovy
@@ -29,14 +29,14 @@ import org.codenarc.util.AstUtil
*
* @author Leboulanger Mickael
*/
-class EC5001 extends AbstractAstVisitorRule {
+class GCI5001 extends AbstractAstVisitorRule {
- String name = 'EC5001'
+ String name = 'GCI5001'
int priority = 2
- Class astVisitorClass = EC5001AstVisitor
+ Class astVisitorClass = GCI5001AstVisitor
}
-class EC5001AstVisitor extends AbstractAstVisitor {
+class GCI5001AstVisitor extends AbstractAstVisitor {
@Override
void visitMethodCallExpression(MethodCallExpression methodCallExpression) {
diff --git a/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-messages.properties b/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-messages.properties
index 4d8ad6b2..2663cd2d 100644
--- a/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-messages.properties
+++ b/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-messages.properties
@@ -1,6 +1,6 @@
-EC5001.description=Using "multiDexEnabled true" goes against the overall reduction of the weight of the apps and hence must be avoided.
-EC5001.description.html=Using "multiDexEnabled true" goes against the overall reduction of the weight of the apps and hence must be avoided.
+GCI5001.description=Using "multiDexEnabled true" goes against the overall reduction of the weight of the apps and hence must be avoided.
+GCI5001.description.html=Using "multiDexEnabled true" goes against the overall reduction of the weight of the apps and hence must be avoided.
ParameterAssignmentInFilterClosure.description=An assignment operator was used on a parameter in a filtering closure. This is usually a typo, and the comparison operator (==) was intended.
ParameterAssignmentInFilterClosure.description.html=An assignment operator was used on a parameter in a filtering closure. This is usually a typo, and the comparison operator (==) was intended.
diff --git a/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-rules.properties b/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-rules.properties
index ec4b6d7d..b8c60f85 100644
--- a/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-rules.properties
+++ b/codenarc-converter/CodeNarc/src/main/resources/codenarc-base-rules.properties
@@ -80,7 +80,7 @@ DuplicateMapLiteral = org.codenarc.rule.dry.DuplicateMapLiteralRule
DuplicateNumberLiteral = org.codenarc.rule.dry.DuplicateNumberLiteralRule
DuplicateSetValue = org.codenarc.rule.basic.DuplicateSetValueRule
DuplicateStringLiteral = org.codenarc.rule.dry.DuplicateStringLiteralRule
-EC5001 = org.codenarc.rule.ecocode.EC5001
+GCI5001 = org.codenarc.rule.ecocode.GCI5001
ElseBlockBraces = org.codenarc.rule.braces.ElseBlockBracesRule
EmptyCatchBlock = org.codenarc.rule.basic.EmptyCatchBlockRule
EmptyClass = org.codenarc.rule.basic.EmptyClassRule
diff --git a/codenarc-converter/CodeNarc/src/main/resources/codenarc-version.txt b/codenarc-converter/CodeNarc/src/main/resources/codenarc-version.txt
index 21bb5e15..1545d966 100644
--- a/codenarc-converter/CodeNarc/src/main/resources/codenarc-version.txt
+++ b/codenarc-converter/CodeNarc/src/main/resources/codenarc-version.txt
@@ -1 +1 @@
-2.2.5
+3.5.0
diff --git a/codenarc-converter/CodeNarc/src/main/resources/rulesets/ecocode.xml b/codenarc-converter/CodeNarc/src/main/resources/rulesets/ecocode.xml
index ef702037..f2b6f6b2 100644
--- a/codenarc-converter/CodeNarc/src/main/resources/rulesets/ecocode.xml
+++ b/codenarc-converter/CodeNarc/src/main/resources/rulesets/ecocode.xml
@@ -7,5 +7,5 @@
EcoCode rule set.