diff options
author | toofar <toofar@spalge.com> | 2024-02-26 19:22:27 +1300 |
---|---|---|
committer | toofar <toofar@spalge.com> | 2024-02-26 19:22:27 +1300 |
commit | 2f3d0161f3b4c2aabbf3f970f8ec25fbac7f7b4d (patch) | |
tree | f655fee0e467b371d078545bcb6c131b5600bcea /qutebrowser | |
parent | 52f5147666fa0111d971ac8b239a9b390d846514 (diff) | |
download | qutebrowser-2f3d0161f3b4c2aabbf3f970f8ec25fbac7f7b4d.tar.gz qutebrowser-2f3d0161f3b4c2aabbf3f970f8ec25fbac7f7b4d.zip |
Update lint etc for dependencies
The lint ones are:
linters (pylint): qutebrowser/completion/completionwidget.py#L440
Consider using 'height = min(height, contents_height)' instead of unnecessary if block
linters (pylint): qutebrowser/browser/webengine/webview.py#L241
Useless suppression of 'no-member'
The no-member one might be due to this change: https://github.com/pylint-dev/pylint/issues/9246
For the importlib-resources one, I'm not sure why it's changed to be
underscore instead of a dash now. But at least it's consistent across
all the requirements files now!
I feel like I've seem this in a previous dependancy update too (for a
different package?) but I can't find that now.
Diffstat (limited to 'qutebrowser')
-rw-r--r-- | qutebrowser/browser/webengine/webview.py | 4 | ||||
-rw-r--r-- | qutebrowser/completion/completionwidget.py | 3 |
2 files changed, 2 insertions, 5 deletions
diff --git a/qutebrowser/browser/webengine/webview.py b/qutebrowser/browser/webengine/webview.py index a6f2ae113..96c0c97e5 100644 --- a/qutebrowser/browser/webengine/webview.py +++ b/qutebrowser/browser/webengine/webview.py @@ -238,9 +238,7 @@ class WebEnginePage(QWebEnginePage): self._set_bg_color() config.instance.changed.connect(self._set_bg_color) if machinery.IS_QT6: - self.certificateError.connect( # pylint: disable=no-member - self._handle_certificate_error - ) + self.certificateError.connect(self._handle_certificate_error) # Qt 5: Overridden method instead of signal @config.change_filter('colors.webpage.bg') diff --git a/qutebrowser/completion/completionwidget.py b/qutebrowser/completion/completionwidget.py index f042be0a1..0f5dc0de9 100644 --- a/qutebrowser/completion/completionwidget.py +++ b/qutebrowser/completion/completionwidget.py @@ -437,8 +437,7 @@ class CompletionView(QTreeView): contents_height = ( self.viewportSizeHint().height() + bar.sizeHint().height()) - if contents_height <= height: - height = contents_height + height = min(height, contents_height) # The width isn't really relevant as we're expanding anyways. return QSize(-1, height) |