Merge pull request #461 from GeneCodeSavvy/fix-pylint-consider-using-enumerate

range replaced with enumerate

Author: tcmitchell

sbol3/toplevel.py

@@ -3,11 +3,11 @@
 import copy
 import math
 import posixpath
+import typing
 import urllib.parse
 import uuid
 import warnings
-from typing import Dict, Callable, Optional, Any
-import typing
+from typing import Any, Callable, Dict, Optional
 
 from . import *
 from .typing import *
@@ -234,8 +234,8 @@ def update_references_traverser(x):
             if v.property_uri not in x._properties:
                 continue
             items = x._properties[v.property_uri]
-            for i in range(len(items)):
-                str_item = str(items[i])
+            for i, item in enumerate(items):
+                str_item = str(item)
                 if str_item in identity_map:
                     new_reference = identity_map[str_item].identity
                     # The item is probably an rdflib.URIRef. We take

setup.cfg

@@ -20,7 +20,6 @@ disable = abstract-class-instantiated,
           implicit-str-concat,
           import-outside-toplevel,
           logging-not-lazy,
-          consider-using-enumerate,
           consider-using-f-string,
           cyclic-import,
           duplicate-code,

test/test_component.py

@@ -148,8 +148,7 @@ def test_cloning_references(self):
         doc2 = sbol3.Document()
         doc2.add(toggle_clone)
         self.assertEqual(len(toggle.constraints), len(toggle_clone.constraints))
-        for i in range(len(toggle.constraints)):
-            c = toggle.constraints[i]
+        for i, c in enumerate(toggle.constraints):
             c_clone = toggle_clone.constraints[i]
             self.assertNotEqual(c.identity, c_clone.identity)
             s = c.subject.lookup()