-
Notifications
You must be signed in to change notification settings - Fork 31
More qflags #171
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
More qflags #171
Conversation
…ion in module Qt3DRender
…e in module Qt3DRender
…in module QtChart
…Orientation in module QtChart
…in module QtChart
…anager.TargetAccessMode in module QtNfc
…reManager.ShareMode in module QtNfc
….AccessMethod in module QtNfc
… module QtBluetooth
…ccessConstraint in module QtBluetooth
…ble.RunFlag in module QtMultimedia
… module QtSerialPort
…nal in module QtSerialPort
…lag in module QtWebEngineCore
…ContextMenuData.MediaFlag in module QtWebEngineWidgets
…ontextMenuData.EditFlag in module QtWebEngineWidgets
…g in module QtWebEngineWidgets
…ormWindowInterface.FeatureFlag in module QtDesigner
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a lot of new files. Perhaps they should be grouped by the module they are for?
tests/test_stubs.py
Outdated
for path in TESTS_DIR.glob('*.py'): | ||
if not path.name.startswith('test_'): | ||
yield path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for path in TESTS_DIR.glob('*.py'): | |
if not path.name.startswith('test_'): | |
yield path | |
yield from TESTS_DIR.glob('test_*.py') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the suggestion changes the code. We want the path not starting with test_
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uh, right... Sorry.
Grouping the files could make sense althought it means spending again some time on the generation scripts. If that's OK for you, I would like to commit them asis and update them by grouping them later in 2022. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure.
Add QFlag related methods to all modules within PyQt5-stubs. This is to be merged after #153 .