-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Python2 only code #89
Comments
This Lines 12 to 24 in 5516142
|
In Python2, f-string syntax was not supported, and Python 2 compatible Lines 1070 to 1075 in 5516142
These can be changed to using f-string syntax: f"""<helics.{self.__class__.__name__}(identifier = "{identifier}", address = "{address}") at {hex(id(self))}>""" |
I believe you don't need these at the top of the file any more: Line 1 in 5516142
|
You can drop Lines 558 to 559 in 5516142
|
It looks like ruff implements pyupgrade checks/fixes, so maybe we could also run this to catch some syntax related updates: https://docs.astral.sh/ruff/rules/#pyupgrade-up |
With Python2 support being dropped in #78, code associated with Python2 syntax can be upgraded to more idiomatic Python3 syntax.
This is a tracking issue for making these changes.
The text was updated successfully, but these errors were encountered: