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

Use "exec()" instead of "exec_()" with PySide6 #1987

Open
rahulporuri opened this issue Mar 9, 2023 · 0 comments
Open

Use "exec()" instead of "exec_()" with PySide6 #1987

rahulporuri opened this issue Mar 9, 2023 · 0 comments

Comments

@rahulporuri
Copy link
Contributor

Like enthought/pyface#1179, we should aim to use exec() with PySide6 instead of exec_() because the latter is deprecated.

C:\Users\rporuri\.edm\envs\<env-name>\lib\site-packages\traitsui\qt4\ui_base.py:320: DeprecationWarning: 'exec_' will be removed in the future. Use 'exec' instead.
  ui.control.exec_()

We see the above deprecation warning when we run our testsuite. The deprecation warning is coming from

ui.control.exec_()

There are a number of similar uses of exec_() in the codebase. For the record, enthought/pyface#1208 is a more or less accepted solution at the moment to handle the use of exec() vs exec_().

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

No branches or pull requests

1 participant