feat!: remove support for python2.7 (#62)

* feat!: remove support for python2.7 Signed-off-by: Dominik Rosiek <drosiek@sumologic.com> * chore: fix lint Signed-off-by: Dominik Rosiek <drosiek@sumologic.com> --------- Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
SumoLogic · Jun 10, 2024 · aaa00c5 · aaa00c5
1 parent f8b8314
commit aaa00c5
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 11 deletions.
diff --git a/.github/workflows/pull_requests.yml b/.github/workflows/pull_requests.yml
@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-20.04
     strategy:
       matrix:
-        python-version: ["2.7", "3.6", "3.7", "3.8", "3.9", "3.10"]
+        python-version: ["3.6", "3.7", "3.8", "3.9", "3.10"]
     steps:
     - uses: actions/checkout@v2
     - name: Set up Python ${{ matrix.python-version }}

diff --git a/requirements-lint.txt b/requirements-lint.txt
@@ -1,5 +1,5 @@
 # This is a separate file because black doesn't run on Python 2, eveb though it
 # supports formatting Python 2 syntax.
-flake8==4.0.1
-black==22.3.0
-isort~=5.10.1
+flake8~=7.0.0
+black~=24.4.2
+isort~=5.13.2
diff --git a/sumologic_collectd_metrics/metrics_converter.py b/sumologic_collectd_metrics/metrics_converter.py
@@ -144,7 +144,7 @@ def convert_to_metrics(data, data_set, sep, extra_dimensions=None):
     metric_dimension = _gen_metric_dimension(data, sep)
     meta_tags = _gen_meta_tags(data)
 
-    for (value, data_type) in zip(data.values, data_set):
+    for value, data_type in zip(data.values, data_set):
         if math.isnan(value):
             continue
 

diff --git a/sumologic_collectd_metrics/metrics_sender.py b/sumologic_collectd_metrics/metrics_sender.py
@@ -168,7 +168,7 @@ def _build_header(self):
             HeaderKeys.x_sumo_host,
             HeaderKeys.x_sumo_category,
         ]
-        for (config_key, header_key) in zip(sumo_config_keys, sumo_header_keys):
+        for config_key, header_key in zip(sumo_config_keys, sumo_header_keys):
             if config_key in config_keys:
                 headers[header_key] = self.conf[config_key]
 

diff --git a/test/collectd/helper.py b/test/collectd/helper.py
@@ -63,7 +63,7 @@ def signalfx_statsd_tags_node():
 
     @staticmethod
     def parse_configs(met_config, configs):
-        for (key, value) in configs.items():
+        for key, value in configs.items():
             node = ConfigNode(getattr(ConfigOptions, key), [value])
             config = CollectdConfig([Helper.url_node(), node])
             met_config.parse_config(config)
diff --git a/test/collectd/values.py b/test/collectd/values.py
@@ -57,7 +57,7 @@ def metrics_str(self, sep="."):
                 sep.join(v for v in (self.type, self.type_instance) if v)
             )
 
-        for (ds_name, ds_type) in zip(self.ds_names, self.ds_types):
+        for ds_name, ds_type in zip(self.ds_names, self.ds_types):
             if not self.meta:
                 metrics.append(
                     "host=%s plugin=%s plugin_instance=%s type=%s type_instance=%s "

diff --git a/test/test_metrics_config.py b/test/test_metrics_config.py
@@ -96,7 +96,7 @@ def test_parse_string_config():
 
     Helper.parse_configs(met_config, configs)
 
-    for (key, value) in configs.items():
+    for key, value in configs.items():
 
         assert met_config.conf[getattr(ConfigOptions, key)] == value
 
@@ -118,7 +118,7 @@ def test_parse_int_config():
 
     Helper.parse_configs(met_config, configs)
 
-    for (key, value) in configs.items():
+    for key, value in configs.items():
 
         assert met_config.conf[getattr(ConfigOptions, key)] == int(value)
 

diff --git a/test/test_metrics_sender.py b/test/test_metrics_sender.py
@@ -120,7 +120,7 @@ def test_post_normal_addition_dimensions_metadata():
         "dimension_tags": ("dim_key1", "dim_val1", "dim_key2", "dim_val2"),
         "meta_tags": ("meta_key1", "meta_val1", "meta_key2", "meta_val2"),
     }
-    for (key, value) in configs.items():
+    for key, value in configs.items():
         node = ConfigNode(getattr(ConfigOptions, key), value)
         config = CollectdConfig([Helper.url_node(), node])
         met_config.parse_config(config)