Skip to content
This repository has been archived by the owner on Aug 9, 2024. It is now read-only.

Commit

Permalink
chore(lint): Fix confeditor so it passes lint
Browse files Browse the repository at this point in the history
* Needed to replace type(...) == type calls with isinstance(...)
    Using type(...) == type instead of isinstance violates lint rule E721.

Signed-off-by: Jason C. Nucciarone <[email protected]>
  • Loading branch information
NucciTheBoss committed Aug 24, 2023
1 parent 1702ec0 commit 2fe34c6
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/utils/confeditor.py
Original file line number Diff line number Diff line change
Expand Up @@ -525,7 +525,7 @@ def auth_alt_types(self) -> Optional[List[str]]:
@auth_alt_types.setter
def auth_alt_types(self, value: Union[str, List[str]]) -> None:
"""Set configuration value for parameter `AuthAltTypes`."""
value = [value] if type(value) == str else value
value = [value] if isinstance(value, str) else value
self._metadata[_SlurmdbdToken.AuthAltTypes] = ",".join(value)

@auth_alt_types.deleter
Expand Down Expand Up @@ -600,7 +600,7 @@ def communication_parameters(self) -> Optional[List[str]]:
@communication_parameters.setter
def communication_parameters(self, value: Union[str, List[str]]) -> None:
"""Set configuration value for parameter `CommunicationParameters`."""
value = [value] if type(value) == str else value
value = [value] if isinstance(value, str) else value
self._metadata[_SlurmdbdToken.CommunicationParameters] = ",".join(value)

@communication_parameters.deleter
Expand Down Expand Up @@ -685,7 +685,7 @@ def debug_flags(self) -> Optional[List[str]]:
@debug_flags.setter
def debug_flags(self, value: Union[str, List[str]]) -> None:
"""Set configuration value for parameter `DebugFlags`."""
value = [value] if type(value) == str else value
value = [value] if isinstance(value, str) else value
for flag in value:
_check_debug_flag(flag)
self._metadata[_SlurmdbdToken.DebugFlags] = ",".join(value)
Expand Down Expand Up @@ -820,7 +820,7 @@ def parameters(self) -> Optional[List[str]]:
@parameters.setter
def parameters(self, value: Union[str, List[str]]) -> None:
"""Set configuration value for parameter `Parameters`."""
value = [value] if type(value) == str else value
value = [value] if isinstance(value, str) else value
self._metadata[_SlurmdbdToken.Parameters] = ",".join(value)

@parameters.deleter
Expand Down Expand Up @@ -855,7 +855,7 @@ def plugin_dir(self) -> Optional[List[str]]:
@plugin_dir.setter
def plugin_dir(self, value: Union[str, List[str]]) -> None:
"""Set configuration value for parameter `PluginDir`."""
value = [value] if type(value) == str else value
value = [value] if isinstance(value, str) else value
self._metadata[_SlurmdbdToken.PluginDir] = ":".join(value)

@plugin_dir.deleter
Expand All @@ -875,7 +875,7 @@ def private_data(self) -> Optional[List[str]]:
@private_data.setter
def private_data(self, value: Union[str, List[str]]) -> None:
"""Set configuration value for parameter `PrivateData`."""
value = [value] if type(value) == str else value
value = [value] if isinstance(value, str) else value
for data in value:
_check_private_data(data)
self._metadata[_SlurmdbdToken.PrivateData] = ",".join(value)
Expand Down

0 comments on commit 2fe34c6

Please sign in to comment.