summaryrefslogtreecommitdiff
path: root/qutebrowser
diff options
context:
space:
mode:
authortoofar <toofar@spalge.com>2024-02-26 19:22:27 +1300
committertoofar <toofar@spalge.com>2024-02-26 19:22:27 +1300
commit2f3d0161f3b4c2aabbf3f970f8ec25fbac7f7b4d (patch)
treef655fee0e467b371d078545bcb6c131b5600bcea /qutebrowser
parent52f5147666fa0111d971ac8b239a9b390d846514 (diff)
downloadqutebrowser-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.py4
-rw-r--r--qutebrowser/completion/completionwidget.py3
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)