Skip to content

Commit

Permalink
Remove requirement on requests-ftp (#51)
Browse files Browse the repository at this point in the history
This module relies on cgi, which was removed in py313 and also most FTP
servers are getting replaced with HTTP anyway
  • Loading branch information
cthoyt authored Dec 2, 2024
1 parent 6e882c7 commit fb07b49
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 5 deletions.
1 change: 0 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ license = { file = "LICENSE" }
requires-python = ">=3.9"
dependencies = [
"requests",
"requests_ftp",
"beautifulsoup4",
"cachier>=2.2.1",
"pystow>=0.1.0",
Expand Down
2 changes: 1 addition & 1 deletion src/bioversions/sources/expasy.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"ExPASyGetter",
]

URL = "ftp://ftp.expasy.org/databases/enzyme/enzuser.txt"
URL = "https://ftp.expasy.org/databases/enzyme/enzuser.txt"


class ExPASyGetter(Getter):
Expand Down
3 changes: 0 additions & 3 deletions src/bioversions/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
import pydantic
import pystow
import requests
import requests_ftp
from bs4 import BeautifulSoup
from cachier import cachier

Expand All @@ -20,8 +19,6 @@
DOCS = os.path.abspath(os.path.join(HERE, os.pardir, os.pardir, "docs"))
IMG = os.path.join(DOCS, "img")

requests_ftp.monkeypatch_session()


class VersionType(enum.Enum):
"""Different types of versions."""
Expand Down

0 comments on commit fb07b49

Please sign in to comment.