Remove augur.utils.read_strains

`augur.utils.read_strains` has been deprecated since January 2024.
Remove it so we stop duplicating `augur.io.read_strains`.

DEPRECATED.md

@@ -16,7 +16,7 @@ Users who have both 'name' and 'strain' fields in their data, and want to favor
 
 ## `augur.utils.read_strains`
 
-*Deprecated in version 24.0.0 (January 2024). Planned for removal March 2024 or after.*
+*Deprecated in version 24.0.0 (January 2024). Removed in version 29.0.0 (February 2025).*
 
 This is part of a [larger effort](https://github.com/nextstrain/augur/issues/1011)
 to formalize Augur's Python API.

augur/utils.py

@@ -6,13 +6,11 @@
 import pandas as pd
 from collections import defaultdict, OrderedDict
 from io import RawIOBase
-from textwrap import dedent
 from .__version__ import __version__
 
 from augur.data import as_file
 from augur.io.file import PANDAS_READ_CSV_OPTIONS, open_file
 from augur.io.sequences import read_single_sequence
-from augur.io.print import print_err
 
 from augur.types import ValidationMode
 from augur.errors import AugurError
@@ -766,13 +764,6 @@ def load_mask_sites(mask_file):
 }
 
 
-def read_strains(*files, comment_char="#"):
-    print_err(dedent("""
-        DEPRECATION WARNING: augur.utils.read_strains is no longer maintained and will be removed in the future.
-        Please use augur.io.read_strains instead."""))
-    return set(read_entries(*files, comment_char=comment_char))
-
-
 def read_entries(*files, comment_char="#"):
     """Reads entries (one per line) from one or more plain text files.
 

tests/functional/mask/variants.vcf.gz_maskTemp

@@ -0,0 +1,5 @@
+MTB_anc	1199
+MTB_anc	1200
+MTB_anc	8321
+MTB_anc	8322
+MTB_anc	8323