fix: remove 'property' from reserved names (#613)

The Python 'property' builtin name should not be reserved.
googleapis · Sep 22, 2020 · 8338a51 · 8338a51
1 parent 6515b77
commit 8338a51
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/gapic/utils/reserved_names.py b/gapic/utils/reserved_names.py
@@ -17,13 +17,11 @@
 import keyword
 
 
-# The filter and map builtins are a historical artifact;
-# they are not used in modern, idiomatic python,
-# nor are they used in the gapic surface.
-# They are too useful to reserve.
+# The exceptions to builtins are frequent and useful.
+# They are explicitly allowed message, module, and field names.
 RESERVED_NAMES = frozenset(
     itertools.chain(
         keyword.kwlist,
-        set(dir(builtins)) - {"filter", "map", "id"},
+        set(dir(builtins)) - {"filter", "map", "id", "property"},
     )
 )