Skip to content

Dap4 package purge #287

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
ndp-opendap wants to merge 13 commits into
base: master
Choose a base branch
from

Merge branch 'master' into dap4-package-purge

fc441d9
Select commit
Loading
Failed to load commit list.
Open

Dap4 package purge #287

Merge branch 'master' into dap4-package-purge
fc441d9
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 26, 2026 in 29s

Quality Gate passed

Issues
22 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 683 in src/opendap/dap4/dmrpp/DmrppJoinNewAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6oAeM0YAVBIObqWe&open=AZze6oAeM0YAVBIObqWe&pullRequest=287

Check warning on line 293 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "aggVarName" which hides the field declared at line 85. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWP&open=AZze6n-NM0YAVBIObqWP&pullRequest=287

Check warning on line 631 in src/opendap/dap4/dmrpp/DmrppJoinNewAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this variable to not match a restricted identifier. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6oAeM0YAVBIObqWd&open=AZze6oAeM0YAVBIObqWd&pullRequest=287

Check warning on line 587 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless assignment to local variable "dimSize". 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWS&open=AZze6n-NM0YAVBIObqWS&pullRequest=287

Check warning on line 216 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWO&open=AZze6n-NM0YAVBIObqWO&pullRequest=287

Check warning on line 587 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "dimSize" local variable. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWT&open=AZze6n-NM0YAVBIObqWT&pullRequest=287

Check warning on line 824 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWM&open=AZze6n-NM0YAVBIObqWM&pullRequest=287

Check warning on line 773 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

"dataURL" is already a string, there's no need to call "toString()" on it. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWX&open=AZze6n-NM0YAVBIObqWX&pullRequest=287

Check warning on line 633 in src/opendap/dap4/dmrpp/DmrppJoinNewAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6oAeM0YAVBIObqWb&open=AZze6oAeM0YAVBIObqWb&pullRequest=287

Check warning on line 759 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWV&open=AZze6n-NM0YAVBIObqWV&pullRequest=287

Check warning on line 865 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWZ&open=AZze6n-NM0YAVBIObqWZ&pullRequest=287

Check warning on line 182 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZ0ICroTPWE6nyE9h9hE&open=AZ0ICroTPWE6nyE9h9hE&pullRequest=287

Check failure on line 119 in src/opendap/dap4/dmrpp/DmrppJoinNewAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change this "try" to a try-with-resources. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6oAeM0YAVBIObqWc&open=AZze6oAeM0YAVBIObqWc&pullRequest=287

Check warning on line 294 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "aggVarElement" which hides the field declared at line 84. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWQ&open=AZze6n-NM0YAVBIObqWQ&pullRequest=287

Check warning on line 182 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWL&open=AZze6n-NM0YAVBIObqWL&pullRequest=287

Check warning on line 765 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWW&open=AZze6n-NM0YAVBIObqWW&pullRequest=287

Check warning on line 814 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this variable to not match a restricted identifier. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWY&open=AZze6n-NM0YAVBIObqWY&pullRequest=287

Check warning on line 3 in src/opendap/dap4/dmrpp/DmrppAggException.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unnecessary import: java.lang classes are always implicitly imported. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6oASM0YAVBIObqWa&open=AZze6oASM0YAVBIObqWa&pullRequest=287

Check warning on line 330 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "aggVarName" which hides the field declared at line 85. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWR&open=AZze6n-NM0YAVBIObqWR&pullRequest=287

Check warning on line 307 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZ0ICroTPWE6nyE9h9hF&open=AZ0ICroTPWE6nyE9h9hF&pullRequest=287

Check failure on line 155 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change this "try" to a try-with-resources. 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWN&open=AZze6n-NM0YAVBIObqWN&pullRequest=287

Check warning on line 758 in src/opendap/dap4/dmrpp/DmrppJoinExistingAggregator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless assignment to local variable "tChunkDimSizesElement". 

See more on https://sonarcloud.io/project/issues?id=opendap-olfs&issues=AZze6n-NM0YAVBIObqWU&open=AZze6n-NM0YAVBIObqWU&pullRequest=287
View more details on SonarQubeCloud