RUF010: Avoid calling str() within an f-string (#2871)

seisman · web-flow · commit 397ef4573bdc · 2023-12-14T17:04:44.000+08:00
diff --git a/pygmt/clib/session.py b/pygmt/clib/session.py
@@ -772,15 +772,15 @@ def _parse_constant(self, constant, valid, valid_modifiers=None):
             )
         if name not in valid:
             raise GMTInvalidInput(
-                f"Invalid constant argument '{name}'. Must be one of {str(valid)}."
+                f"Invalid constant argument '{name}'. Must be one of {valid}."
             )
         if (
             nmodifiers > 0
             and valid_modifiers is not None
             and parts[1] not in valid_modifiers
         ):
             raise GMTInvalidInput(
-                f"Invalid constant modifier '{parts[1]}'. Must be one of {str(valid_modifiers)}."
+                f"Invalid constant modifier '{parts[1]}'. Must be one of {valid_modifiers}."
             )
         integer_value = sum(self[part] for part in parts)
         return integer_value

Original file line number	Diff line number	Diff line change
`@@ -772,15 +772,15 @@ def _parse_constant(self, constant, valid, valid_modifiers=None):`
`772`	`772`	`)`
`773`	`773`	`if name not in valid:`
`774`	`774`	`raise GMTInvalidInput(`
`775`		`- f"Invalid constant argument '{name}'. Must be one of {str(valid)}."`
	`775`	`+ f"Invalid constant argument '{name}'. Must be one of {valid}."`
`776`	`776`	`)`
`777`	`777`	`if (`
`778`	`778`	`nmodifiers > 0`
`779`	`779`	`and valid_modifiers is not None`
`780`	`780`	`and parts[1] not in valid_modifiers`
`781`	`781`	`):`
`782`	`782`	`raise GMTInvalidInput(`
`783`		`- f"Invalid constant modifier '{parts[1]}'. Must be one of {str(valid_modifiers)}."`
	`783`	`+ f"Invalid constant modifier '{parts[1]}'. Must be one of {valid_modifiers}."`
`784`	`784`	`)`
`785`	`785`	`integer_value = sum(self[part] for part in parts)`
`786`	`786`	`return integer_value`