Skip to content

Commit 0b3f8de

Browse files
committed
feat: add ty to dev dependencies and remove deps-ty target
Add ty to both dependency-groups and optional-dependencies dev sections so it's installed as part of the development environment setup. Removed the deps-ty target from Makefile since ty is now installed via uv sync instead of curl scripts. This ensures ty is available for local development and CI without requiring separate installation.
1 parent 7f250d9 commit 0b3f8de

3 files changed

Lines changed: 33 additions & 28 deletions

File tree

Makefile

Lines changed: 2 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ else
1414
RUN :=
1515
endif
1616

17-
.PHONY: help setup sync deps deps-uv deps-ty deps-trunk deps-docker test check protogen
17+
.PHONY: help setup sync deps deps-uv deps-trunk deps-docker test check protogen
1818

1919
.DEFAULT_GOAL := help
2020

@@ -46,7 +46,7 @@ protogen: ## Regenerate protobuf files (requires Python 3.13+)
4646
test: deps-docker ## Run tests
4747
bash scripts/local-test.sh
4848

49-
deps: deps-uv deps-ty deps-trunk deps-docker ## Check/install tool dependencies (set INSTALL_MISSING=true to auto-install)
49+
deps: deps-uv deps-trunk deps-docker ## Check/install tool dependencies (set INSTALL_MISSING=true to auto-install)
5050

5151
deps-uv:
5252
@(command -v uv >/dev/null 2>&1 && command -v uvx >/dev/null 2>&1) || { \
@@ -80,32 +80,6 @@ deps-uv:
8080
fi; \
8181
}
8282

83-
deps-ty:
84-
@command -v ty >/dev/null 2>&1 || { \
85-
if [ "$(INSTALL_MISSING)" = "true" ]; then \
86-
echo "ty not found, installing..."; \
87-
curl -LsSf https://astral.sh/ty/install.sh | sh; \
88-
else \
89-
echo "Error: ty is not installed."; \
90-
echo ""; \
91-
echo "To install ty:"; \
92-
echo ""; \
93-
if [ "$$(uname)" = "Darwin" ]; then \
94-
echo " macOS:"; \
95-
echo " curl -LsSf https://astral.sh/ty/install.sh | sh"; \
96-
elif [ "$$(uname)" = "Linux" ]; then \
97-
echo " Linux:"; \
98-
echo " curl -LsSf https://astral.sh/ty/install.sh | sh"; \
99-
else \
100-
echo " Windows:"; \
101-
echo " powershell -ExecutionPolicy ByPass -c \"irm https://astral.sh/ty/install.ps1 | iex\""; \
102-
fi; \
103-
echo ""; \
104-
echo "Or run: INSTALL_MISSING=true make setup"; \
105-
exit 1; \
106-
fi; \
107-
}
108-
10983
deps-trunk:
11084
@command -v trunk >/dev/null 2>&1 || { \
11185
if [ "$(INSTALL_MISSING)" = "true" ]; then \

pyproject.toml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ dev = [
3939
"pydantic-settings>=2.8.1",
4040
"ruff>=0.8.4",
4141
"shellcheck-py>=0.10.0.1",
42+
"ty>=0.0.8",
4243
"types-grpcio>=1.0.0.20251009",
4344
"types-protobuf>=6.32.1.20251210",
4445
"types-requests>=2.32.0.20241016",
@@ -50,6 +51,7 @@ dev = [
5051
"grpcio-tools>=1.66.2",
5152
"pytest>=8.3.3",
5253
"ruff>=0.8.4",
54+
"ty>=0.0.8",
5355
]
5456

5557

uv.lock

Lines changed: 29 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)