Skip to content

Fix issues detected by sonarqube

Mykhailo.Dalchenko requested to merge fix-issues-detected-by-sonarqube into main

Description

Fix a selected number of issues reported by sonarqube. Not fixed:

  • issues related to aerosol calibrations
  • issues related to throughput
  • issues in tests (minor, non-compliant code)
  • issues in docs

Commits

  • Fix sonar sources and exclusions

  • Fix minor complaints from sonarqube

  • Fixing sonarqube issues in lst_adapter module

  • Fix def value in SQLTableInfo constructor

  • Refactor atomspheric db containers constraints

  • Fix issues reported by sonarqube

  • Fix atmo model version string

  • Make DB password mandatory

  • Fix TypeError message

Related Issue

https://sonar-cta-dpps.zeuthen.desy.de/dashboard?id=cta-computing_dpps_calibrationpipeline_calibpipe_AYvXQiGIgQY5-IeV3T1G

How Has This Been Tested?

Tested in CI

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Merge request reports

Loading