search

spotfiresoftware / spotfire-python

Package for Building Python Extensions to Spotfire®
Other
18 stars 6 forks source link

Change string formatting throughout package to use f-strings where appropriate #14

Closed jorobert closed 3 years ago

jorobert commented 3 years ago

Pylint's standard is now to use f-strings rather than other ways of string formatting, primarily for efficiency and ease of reading. I've adapted all of these where appropriate and put in pylint exceptions, primarily for some strange tuple unpacking where it's just cleaner not to. There should be no change in behavior otherwise.

Also addressed a few other Pylint warnings where we didn't declare file encoding. I set these all to utf-8, assuming that's correct.

jorobert commented 3 years ago

Abandoning PR. Moving to PR #15 due to mistaken branch point for this PR.