From 83f8f4f54795487358adf0f783f6f9ba2eae93cf Mon Sep 17 00:00:00 2001 From: Athanasius Date: Tue, 23 Mar 2021 12:44:12 +0000 Subject: [PATCH] Clean up "not Fleet Carriers" gameversion check 1. No need for the () on the if. 2. Log when we force is_beta = True from this. --- monitor.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/monitor.py b/monitor.py index d0e7b08d..55c9ebea 100644 --- a/monitor.py +++ b/monitor.py @@ -385,7 +385,8 @@ class EDLogs(FileSystemEventHandler): # type: ignore # See below self.is_beta = any(v in entry['gameversion'].lower() for v in ('alpha', 'beta')) # TODO: This is an extra paranoia check, because we don't yet know what will (not) be in the Odyssey alpha string. - if ('Fleet Carriers Update' not in entry['gameversion']): + if 'Fleet Carriers Update' not in entry['gameversion']: + logger.warning(f'Forcing is_beta to True due to gameversion: {entry["gameversion"]}') self.is_beta = True self.cmdr = None