Merge pull request #445 from vladistan/fixes-by-ruff

Fixes proposed by Ruff

Author: ialarmedalien

linkml_runtime/index/object_index.py

@@ -70,7 +70,7 @@ def _index(self, obj: Any, parent_key=None, parent=None):
             return {k: self._index(v, parent_key, parent) for k, v in obj.items()}
         cls_name = type(obj).__name__
         if cls_name in self._class_map:
-            cls = self._class_map[cls_name]
+            self._class_map[cls_name]
             pk_val = self._key(obj)
             self._source_object_cache[pk_val] = obj
             if pk_val not in self._child_to_parent:

linkml_runtime/utils/enumerations.py

@@ -18,11 +18,11 @@ def __setitem__(cls, key, value):
             raise ValueError(f"{cls.__name__} - {key} already assigned")
         cls.__dict__[key] = value
 
-    def __setattr__(self, key, value):
+    def __setattr__(cls, key, value):
         from linkml_runtime.linkml_model.meta import PermissibleValue
 
-        if self._defn.code_set and isinstance(value, PermissibleValue) and value.meaning:
-            print(f"Validating {value.meaning} against {self._defn.code_set}")
+        if cls._defn.code_set and isinstance(value, PermissibleValue) and value.meaning:
+            print(f"Validating {value.meaning} against {cls._defn.code_set}")
         super().__setattr__(key, value)
 
     def __contains__(cls, item) -> bool:

linkml_runtime/utils/ruleutils.py

@@ -136,12 +136,12 @@ def subclass_to_rules(
 
 
 def rule_subsumed_by_class(view: SchemaView, rule, cls: ClassDefinition):
-    induced_slots = view.class_induced_slots(cls.name)
+    view.class_induced_slots(cls.name)
     return False
 
 
 def remove_redundant_rules(view: SchemaView, class_name: ClassDefinitionName):
-    induced_slots = view.class_induced_slots(class_name)
+    view.class_induced_slots(class_name)
     cls = view.get_class(class_name)
     redundant_rules = []
     for rule in cls.rules:

linkml_runtime/utils/schemaops.py

@@ -72,6 +72,6 @@ def roll_down(sv: SchemaView, classes: list[CLASS_NAME] = None, mixins=True, is_
             if is_a and c.is_a is not None:
                 del c.is_a
     for d in dels:
-        d_cls = sv.get_class(d)
+        sv.get_class(d)
         sv.delete_class(d)
     sv.set_modified()