Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: flake8-abp/tests/A201.py

Issue 29340727: Noissue - Added flake8 extension accounting for our coding style and some other stuff (Closed)
Patch Set: Addressed comments, fixed two bugs, use ascii() Created May 9, 2016, 4:47 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « flake8-abp/tests/A111.py ('k') | flake8-abp/tests/A202.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: flake8-abp/tests/A201.py
===================================================================
new file mode 100644
--- /dev/null
+++ b/flake8-abp/tests/A201.py
@@ -0,0 +1,44 @@
+# A201
+global a
+a = 1
+
+
+class Foo:
+ global a
+ # A201
+ global a, b
+ a = 2
+ b = 3
+
+ def foo(self):
+ # A201
+ global b, c
+ b = 4
+
+
+math = None
+Class = None
+func = None
+
+
+def lazy_import():
+ global math
+ if math is None:
+ import math
+ return math
+
+
+def lacy_class():
+ global Class
+ if Class is None:
+ class Class:
+ pass
+ return Class
+
+
+def lazy_function():
+ global func
+ if func is None:
+ def func():
+ pass
+ return func
« no previous file with comments | « flake8-abp/tests/A111.py ('k') | flake8-abp/tests/A202.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld