Add links to noinspection comments
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:
-
Follows new guidelines at https://handbook.gitlab.com/handbook/tools-and-tips/editors-and-ides/jetbrains-ides/code-inspection/ -
Add TODO comments with link if specific JetBrains issues were not found/created. (https://youtrack.jetbrains.com/newIssue went down while I had set aside time to work on this, and I could not open new issues)
-
-
Removes noinspection
comments from areas of code outside of theRemote Development
domain, as they are not currently owned by a team which has committed to maintaining them. -
Add missing noinspection
related to default var/method/class/param/const name length warnings which we want to intentionally suppress.
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.
-
I have evaluated the MR acceptance checklist for this MR.