Skip to content

Commit

Permalink
refactor(get-modflow): don't hard-code available os tags (#2426)
Browse files Browse the repository at this point in the history
Since these are given to change, it seems sufficient to raise (as we do) if a corresponding asset doesn't exist.
  • Loading branch information
wpbonelli authored Jan 26, 2025
1 parent 8dd366f commit 6a8f313
Showing 1 changed file with 1 addition and 13 deletions.
14 changes: 1 addition & 13 deletions flopy/utils/get_modflow.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,15 +35,6 @@
"modflow6-nightly-build": "modflow6_nightly",
}
available_repos = list(renamed_prefix.keys())
available_ostags = [
"linux",
"mac",
"macarm",
"win32",
"win64",
"win64ext",
"win64par",
]
max_http_tries = 3

# Check if this is running from flopy
Expand Down Expand Up @@ -83,9 +74,7 @@ def get_suffixes(ostag) -> Tuple[str, str]:
elif "mac" in ostag:
return "", ".dylib"
else:
raise KeyError(
f"unrecognized ostag {ostag!r}; choose one of {available_ostags}"
)
raise KeyError(f"unrecognized ostag {ostag!r}")


def get_request(url, params={}):
Expand Down Expand Up @@ -699,7 +688,6 @@ def cli_main():
)
parser.add_argument(
"--ostag",
choices=available_ostags,
help="Operating system tag; default is to automatically choose.",
)
parser.add_argument(
Expand Down

0 comments on commit 6a8f313

Please sign in to comment.