Skip to content

Commit ffda8ae

Browse files
committed
[ISSUE 21] delete irrelevant EC69 + delete deprecated EC66 + update sonarqube support to 10.4.1
1 parent 17a116c commit ffda8ae

File tree

12 files changed

+10
-174
lines changed

12 files changed

+10
-174
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,13 +10,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1010
### Added
1111

1212
- [#18](https://github.com/green-code-initiative/ecoCode-python/issues/18) Add support for SonarQube 10.4 "DownloadOnlyWhenRequired" feature
13+
- Add Support for SonarQube 10.4.1
1314

1415
### Changed
1516

1617
- [#17](https://github.com/green-code-initiative/ecoCode-python/issues/17) EC7 - correction setter problem on constructor method
1718

1819
### Deleted
1920

21+
- [#4](https://github.com/green-code-initiative/ecoCode-python/issues/4) Deprecate rule EC66 for Python because not applicable (see details inside issue)
22+
- [#21](https://github.com/green-code-initiative/ecoCode-python/issues/21) Deletion of EC69 for python because not relevant (after analysis)
23+
2024
## [1.4.2] - 2024-01-11
2125

2226
### Changed

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,9 @@ Ready to use binaries are available [from GitHub](https://github.com/green-code-
6363
🧩 Compatibility
6464
------------------
6565

66-
| Plugin version | SonarQube version | Java version |
67-
|----------------|-------------------|--------------|
68-
| 1.4.+ | 9.4.+ LTS to 10.3 | 11 / 17 |
66+
| Plugin version | SonarQube version | Java version |
67+
|----------------|---------------------|--------------|
68+
| 1.4.+ | 9.4.+ LTS to 10.4.1 | 11 / 17 |
6969

7070
> Compatibility table of versions lower than 1.4.+ are available from the
7171
> main [ecoCode repository](https://github.com/green-code-initiative/ecoCode#-plugins-version-compatibility).

docker-compose.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
version: "3.3"
22
services:
33
sonar:
4-
image: sonarqube:10.3.0-community
4+
image: sonarqube:10.4.1-community
55
container_name: sonar_ecocode_python
66
ports:
77
- "9000:9000"

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@
5959
<mockito.version>5.3.1</mockito.version>
6060

6161
<!-- temporary version waiting for real automatic release in ecocode repository -->
62-
<ecocode-rules-specifications.version>1.4.7</ecocode-rules-specifications.version>
62+
<ecocode-rules-specifications.version>1.5.0</ecocode-rules-specifications.version>
6363

6464
<sonar-analyzer-commons.version>2.5.0.1358</sonar-analyzer-commons.version>
6565

src/main/java/fr/greencodeinitiative/python/PythonRuleRepository.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,13 +55,11 @@ public String repositoryKey() {
5555
@Override
5656
public List<Class> checkClasses() {
5757
return Arrays.asList(
58-
AvoidDoubleQuoteCheck.class,
5958
AvoidGettersAndSetters.class,
6059
AvoidGlobalVariableInFunctionCheck.class,
6160
AvoidSQLRequestInLoop.class,
6261
AvoidTryCatchWithFileOpenedCheck.class,
6362
AvoidUnoptimizedVectorImagesCheck.class,
64-
NoFunctionCallWhenDeclaringForLoop.class,
6563
AvoidFullSQLRequest.class,
6664
AvoidListComprehensionInIterations.class,
6765
DetectUnoptimizedImageFormat.class,

src/main/java/fr/greencodeinitiative/python/checks/AvoidDoubleQuoteCheck.java

Lines changed: 0 additions & 46 deletions
This file was deleted.

src/main/java/fr/greencodeinitiative/python/checks/NoFunctionCallWhenDeclaringForLoop.java

Lines changed: 0 additions & 41 deletions
This file was deleted.

src/test/java/fr/greencodeinitiative/python/PythonRuleRepositoryTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ void testMetadata() {
7878

7979
@Test
8080
void testRegistredRules() {
81-
assertThat(repository.rules()).hasSize(11);
81+
assertThat(repository.rules()).hasSize(9);
8282
}
8383

8484
@Test

src/test/java/fr/greencodeinitiative/python/checks/AvoidDoubleQuoteTest.java

Lines changed: 0 additions & 29 deletions
This file was deleted.

src/test/java/fr/greencodeinitiative/python/checks/NoFunctionCallWhenDeclaringForLoopTest.java

Lines changed: 0 additions & 28 deletions
This file was deleted.

0 commit comments

Comments
 (0)