diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 6163117..a7d5653 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -31,7 +31,7 @@ linting:pylint: image: $CONTAINER_REGISTRY:linting stage: linting script: - - pylint --disable=no-else-return,no-else-raise,no-else-continue,unnecessary-comprehension,raise-missing-from,unsubscriptable-object --extension-pkg-whitelist=cairo,gi ./libmat2 ./mat2 + - pylint --disable=no-else-return,no-else-raise,no-else-continue,unnecessary-comprehension,raise-missing-from,unsubscriptable-object,use-dict-literal,unspecified-encoding,consider-using-f-string,use-list-literal --extension-pkg-whitelist=cairo,gi ./libmat2 ./mat2 # Once nautilus-python is in Debian, decomment it form the line below - pylint --disable=no-else-return,no-else-raise,no-else-continue,unnecessary-comprehension,raise-missing-from,unsubscriptable-object --extension-pkg-whitelist=Nautilus,GObject,Gtk,Gio,GLib,gi ./nautilus/mat2.py diff --git a/libmat2/epub.py b/libmat2/epub.py index af059d2..be05562 100644 --- a/libmat2/epub.py +++ b/libmat2/epub.py @@ -108,7 +108,7 @@ class EPUBParser(archive.ZipParser): item.append(uniqid) # items without mandatory content - for name in {'language', 'title'}: + for name in ['language', 'title']: uniqid = ET.Element(self.metadata_namespace + name) item.append(uniqid) break # there is only a single block diff --git a/libmat2/video.py b/libmat2/video.py index b4a3232..ae9e463 100644 --- a/libmat2/video.py +++ b/libmat2/video.py @@ -50,7 +50,7 @@ class AbstractFFmpegParser(exiftool.ExiftoolParser): ret = dict() # type: Dict[str, Union[str, dict]] for key, value in meta.items(): - if key in self.meta_key_value_allowlist.keys(): + if key in self.meta_key_value_allowlist: if value == self.meta_key_value_allowlist[key]: continue ret[key] = value