Skip to content

Commit c5db08b

Browse files
authored
Pin all dependencies (#84)
Our current install process is with pip in `install.sh`. `install.sh` makes for an easy installation process, but using pip can mean that users can get different installations of dbt-mcp. The best way to fix this would be to install the app from the `uv.lock` file, not with pip. However, this would require that users have uv installed. In the future, we can use [uv export](https://github.com/astral-sh/uv/issues/12584?utm_source=chatgpt.com) and install from pylock.toml with pip, but [that isn't supported yet](pypa/pip#13334). In the meantime, pinning all dependencies should help provide for a more consistent install.
1 parent a96d1c6 commit c5db08b

File tree

2 files changed

+11
-10
lines changed

2 files changed

+11
-10
lines changed

pyproject.toml

+5-4
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,13 @@ classifiers = [
1414
"Intended Audience :: Developers",
1515
]
1616
dependencies = [
17+
# Pinning all dependencies because this app is installed with pip
18+
# and we want to have a consistent install as much as possible.
1719
"dbt-sl-sdk[sync]==0.11.0",
18-
# We are using some internal functions, so pinning this dependency.
1920
"mcp[cli]==1.6.0",
20-
"pandas>=2.2.3",
21-
"python-dotenv>=1.0.1",
22-
"requests>=2.32.3",
21+
"pandas==2.2.3",
22+
"python-dotenv==1.0.1",
23+
"requests==2.32.3",
2324
]
2425
[dependency-groups]
2526
dev = [

uv.lock

+6-6
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)