diff --git a/sbol3/toplevel.py b/sbol3/toplevel.py index 7cca28d..a892ed3 100644 --- a/sbol3/toplevel.py +++ b/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 diff --git a/setup.cfg b/setup.cfg index 3cf21b6..be0b274 100644 --- a/setup.cfg +++ b/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, diff --git a/test/test_component.py b/test/test_component.py index 8486d92..846f030 100644 --- a/test/test_component.py +++ b/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()