Make inline "pylint: disable=" notes more clear

Switch ambiguous statements such as "pylint: disable=W0621" with clearer disable statements such as "pylint: disable=redefined-outer-name". I propose that we should be using the clearer form throughout the codebase, so code reviewers know what's going on and coders don't blindly carry over disable statements that no longer apply.

This is also the recommendation made here: https://github.com/edx/edx-platform/blob/master/docs/en_us/developers/source/testing/code-quality.rst
As we discussed in the testeng chat room I plan to enable I0021 (useless-suppression) and clean these and other useless disables up in a separate PR, to keep the big ideas separate.