catalyst / catalyst-moodle-workflows

4 stars 9 forks source link

Consider codechecker_max_warnings should instead just be codechecker_warnings #51

Open brendanheywood opened 2 years ago

brendanheywood commented 2 years ago

Lets say codechecker_max_warnings is set to 10 and you make some change and now there are actually only 9 warnings. In this case I think it should throw an error and ask you to change the number from 10 to 9 so it ratchets down. So the only valid number of warnings is the exact number rather than a max