Skip to content

[repo] Contribution guide #2859

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
chernser wants to merge 2 commits into
base: main
Choose a base branch
from
+212 −111

updated contributiong.md and corrected profile name for coverage

85907ef
Select commit
Loading
Failed to load commit list.
Open

[repo] Contribution guide #2859

updated contributiong.md and corrected profile name for coverage
85907ef
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 10, 2026 in 1m 17s

Quality Gate failed

Failed conditions
71.6% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 37 in jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.time.Instant'. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7Cj_6JKQzmjWbelH&open=AZ6v7Cj_6JKQzmjWbelH&pullRequest=2859

Check warning on line 41 in jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this duplicated import. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7Cj_6JKQzmjWbelJ&open=AZ6v7Cj_6JKQzmjWbelJ&pullRequest=2859

Check warning on line 43 in jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.time.ZoneOffset'. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7Cj_6JKQzmjWbelL&open=AZ6v7Cj_6JKQzmjWbelL&pullRequest=2859

Check failure on line 1126 in client-v2/src/main/java/com/clickhouse/client/api/Client.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7CfB6JKQzmjWbelD&open=AZ6v7CfB6JKQzmjWbelD&pullRequest=2859

Check warning on line 40 in jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this duplicated import. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7Cj_6JKQzmjWbelI&open=AZ6v7Cj_6JKQzmjWbelI&pullRequest=2859

Check warning on line 12 in jdbc-v2/src/test/java/com/clickhouse/jdbc/StatementTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.testng.SkipException'. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7CgB6JKQzmjWbelE&open=AZ6v7CgB6JKQzmjWbelE&pullRequest=2859

Check warning on line 1507 in jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge the previous cases into this one using comma-separated label. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7Cj-6JKQzmjWbelF&open=AZ6v7Cj-6JKQzmjWbelF&pullRequest=2859

Check warning on line 42 in jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'java.time.LocalTime'. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7Cj_6JKQzmjWbelK&open=AZ6v7Cj_6JKQzmjWbelK&pullRequest=2859

Check warning on line 485 in client-v2/src/test/java/com/clickhouse/client/ClientTests.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "random"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7CcE6JKQzmjWbelC&open=AZ6v7CcE6JKQzmjWbelC&pullRequest=2859

Check failure on line 1489 in jdbc-v2/src/main/java/com/clickhouse/jdbc/ResultSetImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 23 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ6v7Cj_6JKQzmjWbelG&open=AZ6v7Cj_6JKQzmjWbelG&pullRequest=2859
View more details on SonarQubeCloud