Skip to content

Commit

Permalink
Removed redundant typer[all] dependency and its usages (#194)
Browse files Browse the repository at this point in the history
`typer` is used for CLI interfaces, though this project has no CLI entry
points, so we keep the number of external dependencies to a minimum.
  • Loading branch information
nfx authored and william-conti committed Sep 15, 2023
1 parent 1860e65 commit bfb924d
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 64 deletions.
3 changes: 3 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
clean:
rm -fr htmlcov .mypy_cache .pytest_cache .ruff_cache .coverage

lint:
hatch run lint:verify

Expand Down
4 changes: 0 additions & 4 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ dependencies = [
"PyYAML>=6.0.0,<7.0.0",

# TODO: remove later
"typer[all]>=0.9.0,<0.10.0",
"ratelimit>=2.2.1,<3.0.0",
"tenacity>=8.2.2,<9.0.0",
]
Expand All @@ -43,9 +42,6 @@ test = [
"pytest-mock>=3.0.0,<4.0.0",
]

[project.scripts]
ucx = "databricks.labs.ucx.__main__:entrypoint"

[project.entry-points.databricks]
runtime = "databricks.labs.ucx.runtime:main"

Expand Down
9 changes: 0 additions & 9 deletions src/databricks/labs/ucx/__main__.py

This file was deleted.

Empty file.
51 changes: 0 additions & 51 deletions src/databricks/labs/ucx/cli/app.py

This file was deleted.

0 comments on commit bfb924d

Please sign in to comment.