Update black duck common version

[crowleySynopsys]

Updated bd-common version to 47.0.1

[blackduck-copilot]

Black Duck Security Report

Merging #135 into master will not change security risk.

Added Components

Clean: 1

Removed Components

Clean: 1

Click here to see full report

[JakeMathews]

Why are we using an extended enum if we don't care about the extended value?

[crowleySynopsys]

Why are we using an extended enum if we don't care about the extended value?

Honestly I wasn't really sure about the Extended value, I was just trying to mimic the implementation of snippet matching since it's also an enum we pass to the scanner. Since we omitted none from the individual file matching enum, I didn't know what to set for the default since I think we essentially want the property to be null if not set to one of the three accepted values but the default parameter is non-null

[JakeMathews]

Why are we using an extended enum if we don't care about the extended value?

Honestly I wasn't really sure about the Extended value, I was just trying to mimic the implementation of snippet matching since it's also an enum we pass to the scanner. Since we omitted none from the individual file matching enum, I didn't know what to set for the default since I think we essentially want the property to be null if not set to one of the three accepted values but the default parameter is non-null

You don't need to supply a default value to nullable properties. I think what you want to use is NullableEnumProperty.

[taikuukaits]

I think he does want to use extended enum with NONE. Because the only extended enum value is NONE at the moment he doesn't actually care about it's value. I think he had it correct. We don't want "not-present" to be none. We want a default value of NONE just like snippet matching.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
1 Code Smell

33.3% Coverage
0.0% Duplication