Skip to content

Release: promote Solr changes to main for TEST deployment#1982

Closed
Oluwapizzy wants to merge 28 commits into
mainfrom
feature-solr-merged
Closed

Release: promote Solr changes to main for TEST deployment#1982
Oluwapizzy wants to merge 28 commits into
mainfrom
feature-solr-merged

Merge branch 'main' into feature-solr-merged

3b83d03
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jun 16, 2026 in 59s

Annotations

Check warning on line 195 in jobs/sftp-nuans-report/sftp_nuans_report/notebook/generate_files.ipynb

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this assignment to local variable 'gz_file'; the value is never used.

See more on https://sonarcloud.io/project/issues?id=bcgov_namex&issues=AZ7Sidmi1nN34RRw49_X&open=AZ7Sidmi1nN34RRw49_X&pullRequest=1982

Check warning on line 509 in api/namex/resources/requests.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace set constructor call with a set comprehension.

See more on https://sonarcloud.io/project/issues?id=bcgov_namex&issues=AZ7Sguj_EdyOuX9cla1r&open=AZ7Sguj_EdyOuX9cla1r&pullRequest=1982

Check warning on line 142 in api/namex/services/solr/solr_helpers.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this commented out code.

See more on https://sonarcloud.io/project/issues?id=bcgov_namex&issues=AZ7Sguk4EdyOuX9cla1v&open=AZ7Sguk4EdyOuX9cla1v&pullRequest=1982

Check failure on line 96 in api/namex/services/solr/solr_helpers.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 22 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=bcgov_namex&issues=AZ7Sguk4EdyOuX9cla1u&open=AZ7Sguk4EdyOuX9cla1u&pullRequest=1982

Check failure on line 548 in api/namex/resources/requests.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "logging.exception()" instead.

See more on https://sonarcloud.io/project/issues?id=bcgov_namex&issues=AZ7Sguj_EdyOuX9cla1s&open=AZ7Sguj_EdyOuX9cla1s&pullRequest=1982

Check failure on line 89 in api/namex/services/solr/solr_helpers.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "word" to "self" or add the missing "self" parameter.

See more on https://sonarcloud.io/project/issues?id=bcgov_namex&issues=AZ7Sguk4EdyOuX9cla1t&open=AZ7Sguk4EdyOuX9cla1t&pullRequest=1982

Check warning on line 35 in solr-feeder/solr_feeder/solr.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the unused local variable "token_err" with "_".

See more on https://sonarcloud.io/project/issues?id=bcgov_namex&issues=AZ7Sgun6EdyOuX9cla1w&open=AZ7Sgun6EdyOuX9cla1w&pullRequest=1982