Use the {} constructor instead of dict()

Author: sminot

cirro/clients/s3.py

@@ -35,8 +35,8 @@ class S3Client:
     def __init__(self, creds_getter: Callable[[], AWSCredentials], checksum_method: str = None):
         self._creds_getter = creds_getter
         self._client = self._build_session_client()
-        self._upload_args = dict(ChecksumAlgorithm=checksum_method)
-        self._download_args = dict(ChecksumMode='ENABLED') if checksum_method else dict()
+        self._upload_args = {"ChecksumAlgorithm": checksum_method}
+        self._download_args = {"ChecksumMode": 'ENABLED'} if checksum_method else {}
 
     def get_aws_client(self):
         return self._client

cirro/helpers/form.py

@@ -11,13 +11,13 @@ class FormBuilder:
     def __init__(self):
 
         # Contents will be written out as the form
-        self.form = dict(
-            ui=OrderedDict(),
-            form=OrderedDict(
+        self.form = {
+            "ui": OrderedDict(),
+            "form": OrderedDict(
                 type="object",
                 properties=OrderedDict()
             )
-        )
+        }
 
         # Used to make sure that no keys are repeated
         self.used_keys = set()
@@ -28,7 +28,7 @@ def __init__(self):
         # Store the params which will be populated either by:
         # a) While building the form, the optional `test_value` field will be used
         # b) While running non-interactively, it will use the values read from $PW_NOTEBOOK_DATA
-        self.params = dict()
+        self.params = {}
 
     def add_param(
         self,
@@ -65,7 +65,7 @@ def add_param(
         assert type in self.PARAM_TYPES, msg
 
         # Start building the item in the form
-        item = dict(type=type)
+        item = {"type": type}
 
         # Populate the test value
         if test_value is not None:
@@ -109,11 +109,11 @@ def add_section(self, title: str = None, description: str = None):
         self.pointer = section_name
 
         # Add the section
-        self.form['form']['properties'][section_name] = dict(
-            title=title,
-            description=description,
-            properties=OrderedDict()
-        )
+        self.form['form']['properties'][section_name] = {
+            "title": title,
+            "description": description,
+            "properties": OrderedDict()
+        }
 
     def _new_section_name(self):
         """Internal method to pick a new section name."""

cirro/services/dataset.py

@@ -342,12 +342,12 @@ def validate_folder(
             ]
         ]
 
-        return dict(
-            ds_files_matching=ds_files_matching,
-            ds_files_notmatching=ds_files_notmatching,
-            ds_files_missing=ds_files_missing,
-            local_only_files=local_only_files
-        )
+        return {
+            "ds_files_matching": ds_files_matching,
+            "ds_files_notmatching": ds_files_notmatching,
+            "ds_files_missing": ds_files_missing,
+            "local_only_files": local_only_files
+        }
 
     def file_is_valid(self, ds_file: File, local_file: Path) -> bool:
         try: