Skip to content

Add links to noinspection comments

Chad Woolley requested to merge caw-add-links-to-noinspection-comments into master

What does this MR do and why?

Adds links to noinspection comments.

See https://handbook.gitlab.com/handbook/tools-and-tips/editors-and-ides/jetbrains-ides/code-inspection/why-are-there-noinspection-comments/ for more explanation.

Note that this is a work in progress. This MR is just an initial pass to get things cleaned up, but it will be on ongoing effort.

We have already received a wonderful prompt response from JetBrains on all of the issues we opened or commented on, so I need to follow up on those, but we want to get this first pass merged ASAP since it touches a lot of files and we want to avoid ongoing merge conflicts

TASKS:

Related: Add jetbrains issues, fix some links (gitlab-com/content-sites/handbook!251 - merged)

Non-Goals

This MR does not clean up frontend-related inspection warnings. That will be done in a separate MR.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Chad Woolley

Merge request reports