Skip to content

Commit

Permalink
Remove SqlAlchemy dependency pinning
Browse files Browse the repository at this point in the history
Summary: Since we have resolved the SQLAlchemy 2.0 dependency issue as part of T200894264, we should be good to remove the pinning

Differential Revision: D65429486
  • Loading branch information
paschai authored and facebook-github-bot committed Nov 4, 2024
1 parent 242fb43 commit 927b3a9
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,6 @@
"numpy<2.0",
]

MYSQL_REQUIRES = ["SQLAlchemy==1.4.17"]

NOTEBOOK_REQUIRES = ["jupyter"]

UNITTEST_MINIMAL_REQUIRES = [
Expand All @@ -60,9 +58,7 @@
"torchx", # For torchx unit tests.
]

UNITTEST_REQUIRES = (
DEV_REQUIRES + MYSQL_REQUIRES + NOTEBOOK_REQUIRES + UNITTEST_MINIMAL_REQUIRES
)
UNITTEST_REQUIRES = DEV_REQUIRES + NOTEBOOK_REQUIRES + UNITTEST_MINIMAL_REQUIRES

TUTORIAL_REQUIRES = UNITTEST_REQUIRES + [
"ray", # Required for building RayTune tutorial notebook.
Expand Down Expand Up @@ -114,7 +110,6 @@ def setup_package() -> None:
},
extras_require={
"dev": DEV_REQUIRES,
"mysql": MYSQL_REQUIRES,
"notebook": NOTEBOOK_REQUIRES,
"unittest": UNITTEST_REQUIRES,
"unittest_minimal": UNITTEST_MINIMAL_REQUIRES,
Expand Down

0 comments on commit 927b3a9

Please sign in to comment.