It is best practice to use constants instead of magic numbers. Please replace the severity values in Acquia SPI module with constants to avoid magic numbers. This will make it easier for modules to implement custom insight checks.

Comments

Dane Powell’s picture

Dane Powell CreditAttribution: Dane Powell at Acquia commented
Status: Active » Closed (won't fix)

This branch of Acquia Connector is no longer supported, per the version policy on the project homepage. As such, I'm tentatively closing this issue.

If this issue still applies to a supported branch (currently 7.x-3.x, 8.x-1.x, or 8.x-2.x), please reopen and select the new target version. Thanks!