Skip to content

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

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

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

Merge pull request #1981 from Oluwapizzy/feature/test-set-nr-state

9a2c738
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jun 16, 2026 in 42s

Quality Gate passed

Issues
7 New issues
1 Accepted issue

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

See analysis details on SonarQube Cloud

Annotations

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 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

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 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 62 in jobs/bad-designation-notifier/src/bad_designation_notifier/services/utils.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=AZ7Sguh1EdyOuX9cla1q&open=AZ7Sguh1EdyOuX9cla1q&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 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
View more details on SonarQubeCloud