BLD/STY: use black 23+, I guess

Or we can just pin it if this still doesn't work
fedarko · Jun 22, 2023 · b8dffdc · b8dffdc
1 parent 3a6a80c
commit b8dffdc
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 10 deletions.
diff --git a/.github/workflows/standalone.yml b/.github/workflows/standalone.yml
@@ -33,7 +33,7 @@ jobs:
         run: pip install -e .[dev]
 
       - name: Install Black, also (no QIIME 2 to worry about conflicts with)
-        run: pip install "black >= 22.3.0"
+        run: pip install "black >= 23.0.0"
 
       - name: Set up Node.js
         uses: actions/setup-node@v1

diff --git a/qurro/_df_utils.py b/qurro/_df_utils.py
@@ -657,7 +657,7 @@ def vibe_check(
         "log-ratios outside of the Qurro visualization interface."
     ).format(safe_range[0])
 
-    for (df, df_name) in (
+    for df, df_name in (
         (table_sdf, "feature table"),
         (feature_ranks, "feature rankings data"),
     ):

diff --git a/qurro/q2/_visualizer_utils.py b/qurro/q2/_visualizer_utils.py
@@ -21,7 +21,6 @@ def create_q2_visualization(
     extreme_feature_count,
     debug,
 ):
-
     # Same thing as in the standalone version of Qurro -- only show debug
     # messages if explicitly requested with --(p-)debug. As with there, this is
     # inspired by https://stackoverflow.com/a/14098306/10730311.

diff --git a/qurro/tests/test_df_utils.py b/qurro/tests/test_df_utils.py
@@ -358,7 +358,6 @@ def test_remove_empty_samples_and_features_allempty():
 
 
 def test_print_if_dropped(capsys):
-
     table, metadata, ranks = get_test_data()
 
     # Neither of these should result in anything being printed
@@ -418,7 +417,6 @@ def test_print_if_dropped(capsys):
 
 
 def test_merge_feature_metadata():
-
     ranks = DataFrame({"R1": [1, 2], "R2": [2, 1]}, index=["F1", "F2"])
     fm = DataFrame(
         {"FM1": [None, None], "FM2": [1, 2], "FM3": [8, 7]}, index=["F1", "F2"]
@@ -468,7 +466,6 @@ def test_merge_feature_metadata():
 
 
 def test_sparsify_count_dict():
-
     # Test that it works in basic case
     test_cts = {
         "Feature 1": {"Sample 1": 0, "Sample 2": 3, "Sample 3": 0},
@@ -497,7 +494,6 @@ def test_sparsify_count_dict():
 
 
 def test_check_column_names():
-
     _, sm, fr = get_test_data()
     fm = fr.copy()
     fm.columns = ["FM1", "FM2"]
@@ -833,7 +829,6 @@ def verify_spc_data_integrity(output_feature_data, initial_feature_data):
 
 
 def test_add_sample_presence_count_basic():
-
     # NOTE: for reference, the get_test_data() table initially looks like this:
     # "Sample1": [1, 2, 3, 4, 5, 6, 7, 8],
     # "Sample2": [8, 7, 6, 5, 4, 3, 2, 1],

diff --git a/qurro/tests/test_json_utils.py b/qurro/tests/test_json_utils.py
@@ -126,7 +126,6 @@ def test_get_jsons():
 
 
 def test_plot_jsons_equal():
-
     assert plot_jsons_equal(None, None)
     assert plot_jsons_equal({}, {})
     assert not plot_jsons_equal({"a": "b"}, {"a": "c"})

diff --git a/qurro/tests/testing_utilities.py b/qurro/tests/testing_utilities.py
@@ -357,7 +357,6 @@ def validate_sample_plot_json(
     # filtered out.
     sample_metadata = replace_nan(read_metadata_file(metadata_loc))
     for sample in sample_json["datasets"][dn]:
-
         sample_id = sample["Sample ID"]
 
         for metadata_col in sample_metadata.columns: