-
Notifications
You must be signed in to change notification settings - Fork 538
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run policheck as part of code analysis job #4882
Conversation
Looks like 27 issues were found but the report stage was still green. We may need to create a suppressions list for this task. I'll see if I can get this to fail when issues are detected: |
Example of a build with failing PoliCheck results - https://github.com/xamarin/xamarin-android/runs/823241298 We should be able to merge this after the yaml-templates PR is merged. |
|
Context: https://github.com/xamarin/yaml-templates/pull/42
PoliCheck has been added to our code analysis job. Initial results from
running this tool reported 27 failures, which are all present in
external sources and have been added to an exclusion list.
All code anaylsis steps and reporting for the xamarin-android pipeline
previously lived in yet another template (xa-static-analysis/v2.yml),
but the content of that file is no longer well suited for a template.
Rather than creating another version of this template in the
yaml-templates repo, the core logic has been moved here.