Are the lines needed? By adding them, we make the test a little more brittle in my opinion. If a line was added before line 24, that change will trigger the suspicious_enum_value check, while CO2_VOLUME is unchanged, resulting in a puzzled developer. 😉
Especially the later suppression in line 234 is running that risk.
Discuss what suppressions format is required.
Are the lines needed? By adding them, we make the test a little more brittle in my opinion. If a line was added before line 24, that change will trigger the suspicious_enum_value check, while CO2_VOLUME is unchanged, resulting in a puzzled developer. 😉
Especially the later suppression in line 234 is running that risk.
Discuss what suppressions format is required.