Enable disabled pylint rules pt.1

.pylintrc

@@ -68,25 +68,19 @@ confidence=
 # --disable=W".
 disable=arguments-renamed,              # TODO: investigate / re-enable
         bad-mcs-classmethod-argument,   # TODO: investigate / re-enable
-        bad-continuation, bad-whitespace # differences of opinion with black
-        consider-iterating-dictionary,  # TODO: investigate / re-enable
-        consider-using-dict-items,      # TODO: investigate / re-enable
+        bad-continuation, bad-whitespace, # differences of opinion with black
         consider-using-f-string,        # TODO: investigate / re-enable
         consider-using-with,            # TODO: investigate / re-enable
-        differing-param-doc,            # TODO: investigate / re-enable
         docstring-first-line-empty,     # TODO: investigate / re-enable
         duplicate-code,                 # too verbose
-        f-string-without-interpolation, # TODO: investigate / re-enable
         fixme,                          # avoid that to do annotations show up as warnings
         invalid-name,                   # TODO: investigate / re-enable
         missing-param-doc,              # TODO: investigate / re-enable (issues with kwargs docs)
-        missing-type-doc,               # TODO: investigate / re-enable
         no-else-return,                 # relax "elif" after a clause with a return
         no-member,                      # TODO: investigate / re-enable
         no-self-use,                    # too verbose
         protected-access,               # we don't strictly follow the public vs. private convention
         raise-missing-from,             # TODO: investigate / re-enable
-        redundant-returns-doc,          # TODO: investigate / re-enable
         super-with-arguments,           # TODO: investigate / re-enable
         too-few-public-methods,         # too verbose
         too-many-ancestors,             # too verbose
@@ -100,8 +94,6 @@ disable=arguments-renamed,              # TODO: investigate / re-enable
         too-many-statements,            # too verbose
         unnecessary-pass,               # allow for methods with just "pass", for clarity
         unspecified-encoding,           # TODO: investigate / re-enable
-        use-dict-literal,               # TODO: investigate / re-enable
-        use-list-literal,               # TODO: investigate / re-enable
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option

qiskit_ibm_runtime/api/client_parameters.py

@@ -37,8 +37,10 @@ def __init__(
         """ClientParameters constructor.
 
         Args:
+            auth_type: Authentication type. ``cloud`` or ``legacy``.
             token: IBM Quantum API token.
             url: IBM Quantum URL (gets replaced with a new-style URL with hub, group, project).
+            instance: Service instance to use.
             proxies: Proxy configuration.
             verify: If ``False``, ignores SSL certificates errors.
         """

qiskit_ibm_runtime/api/clients/base.py

@@ -223,10 +223,7 @@ def stream(
 
                 self._current_retry += 1
                 if self._current_retry > retries:
-                    error_message = (
-                        "Max retries exceeded: Failed to establish a "
-                        f"websocket connection."
-                    )
+                    error_message = "Max retries exceeded: Failed to establish a websocket connection."
                     if self._error:
                         error_message += f" Error: {self._error}"
 

qiskit_ibm_runtime/api/rest/program.py

@@ -64,11 +64,7 @@ def make_private(self) -> None:
         self.session.put(url)
 
     def delete(self) -> None:
-        """Delete this program.
-
-        Returns:
-            JSON response.
-        """
+        """Delete this program."""
         url = self.get_url("self")
         self.session.delete(url)
 

qiskit_ibm_runtime/program/program_template.py

@@ -40,7 +40,7 @@ def main(backend: ProgramBackend, user_messenger: UserMessenger, **kwargs) -> An
     Args:
         backend: Backend for the circuits to run on.
         user_messenger: Used to communicate with the program user.
-        kwargs: User inputs.
+        **kwargs: User inputs.
 
     Returns:
         The final result of the runtime program.

qiskit_ibm_runtime/utils/utils.py

@@ -41,6 +41,9 @@ def get_runtime_api_base_url(url: str, instance: str) -> str:
     Args:
         url: The URL.
         instance: The instance.
+
+    Returns:
+        Runtime API base URL
     """
 
     # legacy: no need to resolve runtime API URL

test/mock/fake_account_client.py

@@ -49,6 +49,7 @@ def __init__(
         """Initialize a fake account client.
 
         Args:
+            hgp: Hub/group/project to use.
             num_backends: Number of backends. Ignored if ``specs`` is specified.
             specs: Backend specs. This is a dictionary of overwritten backend
                 configuration / status. For example::