oxsecurity / megalinter

🦙 MegaLinter analyzes 50 languages, 22 formats, 21 tooling formats, excessive copy-pastes, spelling mistakes and security issues in your repository sources with a GitHub Action, other CI tools or locally.
https://megalinter.io
GNU Affero General Public License v3.0
1.96k stars 238 forks source link

Azure DevOps PR status incorrect when using FAIL_IF_UPDATED_SOURCES #4210

Closed lukelloyd1985 closed 3 weeks ago

lukelloyd1985 commented 3 weeks ago

Describe the bug Using the FAIL_IF_UPDATED_SOURCES variable is meant to fail MegaLinter if there are auto fixed files (which it does). However the Azure PR comment shows as success:

To Reproduce Steps to reproduce the behavior:

  1. Configure a DevOps pipeline with the FAIL_IF_UPDATED_SOURCES variable
  2. Raise a PR with a file known to require an auto fix
  3. Voila, MegaLinter job should fail (as per screenshot) but PR comment with show "Success" and be raised resolved

Expected behavior PR comment to show the MegaLinter failure and the comment to be in an active state

Screenshots image

image

nvuillam commented 3 weeks ago

@lukelloyd1985 on it :)