Linked issue in fedora/infra noggin: https://github.com/fedora-infra/noggin/issues/101 with the linked PR: https://github.com/fedora-infra/noggin/pull/126
Sonarqube needs to be added to the automated testing pipeline, and needs a GitHub fedora/infra admin to enable the webhook. This will give us the ability to test the security and code quality prior to the PR being merged.
<img alt="Screenshot_from_2020-04-02_11-34-44.png" src="/fedora-infrastructure/issue/raw/files/5d4e8723111132b83c4bd26fe3016a01ebcd16a9c7aeb71eee10abb030c3f87d-Screenshot_from_2020-04-02_11-34-44.png" />
Metadata Update from @cverna: - Issue assigned to cverna
Metadata Update from @cverna: - Issue priority set to: Waiting on Assignee (was: Needs Review) - Issue tagged with: low-trouble, medium-gain
I have created https://sonarcloud.io/dashboard?id=fedora-infra_noggin and that should automatically analyse the PR from now.
Let us know if that does not work.
Metadata Update from @cverna: - Issue close_status updated to: Fixed - Issue status updated to: Closed (was: Open)
Login to comment on this ticket.