Apply @dlstadther's suggestions

spotify · Oct 21, 2023 · db92ba7 · db92ba7
1 parent bda147c
commit db92ba7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/luigi/contrib/mongodb.py b/luigi/contrib/mongodb.py
@@ -36,7 +36,7 @@ def __init__(self, mongo_client, index, collection):
         self._collection = collection
 
     def __str__(self):
-        return '%s/%s' % (self._index, self._collection)
+        return f'{self._index}/{self._collection}'
 
     def get_collection(self):
         """

diff --git a/luigi/contrib/sqla.py b/luigi/contrib/sqla.py
@@ -190,7 +190,7 @@ def __init__(self, connection_string, target_table, update_id, echo=False, conne
         self.marker_table_bound = None
 
     def __str__(self):
-        return self.marker_table_bound or self.connection_string
+        return self.target_table
 
     @property
     def engine(self):

diff --git a/luigi/worker.py b/luigi/worker.py
@@ -187,7 +187,7 @@ def run(self):
                     if not self.check_complete(dep):
                         nonexistent_outputs = [output for output in dep.output() if not output.exists()]
                         if nonexistent_outputs:
-                            missing.append('%s (%s)' % (dep.task_id, ', '.join(map(str, nonexistent_outputs))))
+                            missing.append(f'{dep.task_id} ({", ".join(map(str, nonexistent_outputs))})')
                         else:
                             missing.append(dep.task_id)
                 if missing: