Skip to content
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

[py2py3] Apply changes from pylint --py3k to files involved in #9818 #10167

Closed
mapellidario opened this issue Dec 10, 2020 · 1 comment
Closed

Comments

@mapellidario
Copy link
Member

Impact of the new feature

Files involved in #9818

Is your feature request related to a problem? Please describe.

#9818 was not checked against pylint --py3k. Its report is

************* Module src.python.WMCore.WMException
W:117,12: Indexing exceptions will not work on Python 3 (indexing-exception)

We should then modernize WMCore/WMException following pylint doc:

Indexing exceptions will not work on Python 3 Indexing exceptions will not work on Python 3. Use exception.args[index] instead.

Describe the solution you'd like

Simply change the way that the exception arguments are accessed.

Describe alternatives you've considered

There is not much we ccan do if we want to run the same code in python3

Additional context

Follow-up to #9818

@mapellidario
Copy link
Member Author

Fixed by #10187, which superseded #10168

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant