STYLE: Additional flake8 checks
 Currently we ignore some flake8 checks:

Checks that (currently) we're okay with skipping
-  E402,  # module level import not at top of file
-  E731,  # do not assign a lambda expression, use a def
-  W503   # line break before binary operator

Checks that should be run 
- [x]  E741,  # do not use variables named 'l', 'O', or 'I' #22795 (#23131)
- [x] C405,  # Unnecessary (list/tuple) literal - rewrite as a set literal.
- [x] C406   # Unnecessary (list/tuple) literal - rewrite as a dict literal.
- [x] C408,  # Unnecessary (dict/list/tuple) call - rewrite as a literal.
- [x] C409,  # Unnecessary (list/tuple) passed to tuple() - (remove the outer call to tuple()/rewrite as a tuple literal).
- [x] C410   # Unnecessary (list/tuple) passed to list() - (remove the outer call to list()/rewrite as a list literal).

Ideally, we should continue to lint our codebase with reasonable flake8 checks.