AL-Go
AL-Go copied to clipboard
Enable blocking pull requests that add new warnings
Things to consider for feedback:
- Invoking the check for warnings as the last thing in RunPipeline.ps1. Could move to a step in _BuildALGoProject.yaml, but not sure how value it adds?
- Should the setting be named 'checkForNewWarnings', or 'blockNewWarnings', etc?
- We could optimize runtime by only comparing warnings when the number of warnings from baseline has changed, with the risk of ignoring new warnings if introduced at the same time as other warnings are removed.
- Warnings are only stored in the log if the setting 'enableCodeCop' is set to true. Should we throw an error if checkForNewWarnings is $true but enableCodeCop is $false?
- A new compiler version could add new warnings. In this case, a new 'official' build has to run before a PR can be completed.