Show diff in CI and cleanup pylintrc (#33704)

azure-pipelines-ci.yml

@@ -74,11 +74,11 @@ stages:
       displayName: 'Run yamllint'
     - script: |
         . venv/bin/activate
-        pre-commit run pyupgrade --all-files
+        pre-commit run pyupgrade --all-files --show-diff-on-failure
       displayName: 'Run pyupgrade'
     - script: |
         . venv/bin/activate
-        pre-commit run prettier --all-files
+        pre-commit run prettier --all-files --show-diff-on-failure
       displayName: 'Run prettier'
   - job: 'Validate'
     pool:

pylintrc

@@ -18,13 +18,11 @@ good-names=id,i,j,k,ex,Run,_,fp
 # cyclic-import - doesn't test if both import on load
 # abstract-class-little-used - prevents from setting right foundation
 # unused-argument - generic callbacks and setup methods create a lot of warnings
-# global-statement - used for the on-demand requirement installation
 # redefined-variable-type - this is Python, we're duck typing!
 # too-many-* - are not enforced for the sake of readability
 # too-few-* - same as too-many-*
 # abstract-method - with intro of async there are always methods missing
 # inconsistent-return-statements - doesn't handle raise
-# import-outside-toplevel - TODO
 # too-many-ancestors - it's too strict.
 # wrong-import-order - isort guards this
 disable=