-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Dropping __future__ from cmssw #46762
Comments
FYI @cms-sw/all-l2 |
cms-bot internal usage |
A new Issue was created by @smuzaffar. @Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign core |
New categories assigned: core @Dr15Jones,@makortel,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
@smuzaffar according to https://docs.python.org/3/library/__future__.html#module-contents, I do not think |
thanks @kpedro88 , you are right |
We have many python files (over 900) with statements like [a] which we had added for python2/3 compatibility. As CMSSW only supports python3, so I suggest to remove all these lines.
We will open PRs (one per category) to remove these statements.
[a]
The text was updated successfully, but these errors were encountered: