Skip to content

fix: sonarqube wildcard issue

Derek Zhang requested to merge derekxz-code-quality into master

Type of change

  • Bug Fix
  • Feature

Please provide link to gitlab issue or ADR(Architecture Decision Record)

Does this introduce a change in the core logic?

  • [YES/NO]
  • NO

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • Google Cloud
  • IBM

Does this introduce a breaking change?

  • [YES/NO] NO - it shouldn't

What is the current behavior?

sonarqube raise issue of using wildcard type

What is the new/expected behavior?

using a wrapper for the type, sonarqube passes

Have you added/updated Unit Tests and Integration Tests?

Any other useful information

changed return type of testCrc from wildcard to a wrapper class

Merge request reports