From ce4a6ff898138fdeb41896a8aa35db2e31505284 Mon Sep 17 00:00:00 2001 From: Athanasius Date: Thu, 22 Dec 2022 16:54:03 +0000 Subject: [PATCH] plugins/eddn: Remove the 'default values' from should_retrn & new_data In *this* case the variables *are* used in the scope so become bound, so we can get away with bare type declaration. --- plugins/eddn.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/plugins/eddn.py b/plugins/eddn.py index 33ee1802..3cd2d819 100644 --- a/plugins/eddn.py +++ b/plugins/eddn.py @@ -635,8 +635,8 @@ class EDDN: :param data: a dict containing the starport data :param is_beta: whether or not we're currently in beta mode """ - should_return: bool = False - new_data: Dict[str, Any] = {} + should_return: bool + new_data: Dict[str, Any] should_return, new_data = killswitch.check_killswitch('capi.request./market', {}) if should_return: logger.warning("capi.request./market has been disabled by killswitch. Returning.") @@ -768,8 +768,8 @@ class EDDN: :param data: dict containing the outfitting data :param is_beta: whether or not we're currently in beta mode """ - should_return: bool = False - new_data: Dict[str, Any] = {} + should_return: bool + new_data: Dict[str, Any] should_return, new_data = killswitch.check_killswitch('capi.request./shipyard', {}) if should_return: logger.warning("capi.request./shipyard has been disabled by killswitch. Returning.") @@ -836,8 +836,8 @@ class EDDN: :param data: dict containing the shipyard data :param is_beta: whether or not we are in beta mode """ - should_return: bool = False - new_data: Dict[str, Any] = {} + should_return: bool + new_data: Dict[str, Any] should_return, new_data = killswitch.check_killswitch('capi.request./shipyard', {}) if should_return: logger.warning("capi.request./shipyard has been disabled by killswitch. Returning.")