Skip to content

Commit 21504ac

Browse files
kirankashyapKiran KashyapSteveSandersonMSCopilot
authored
Fix justfile install 585 (#634)
* Add per-language install recipes (#585) * fix #585 * Revert unnecessary changes in install recipes - Restore --ignore-scripts flag for test harness npm ci - Remove extraneous uv venv from Python install (uv run manages venvs) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Kiran Kashyap <kiran.kashyap.ds@example.com> Co-authored-by: Steve Sanderson <SteveSandersonMS@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 1a94402 commit 21504ac

File tree

1 file changed

+25
-6
lines changed

1 file changed

+25
-6
lines changed

justfile

Lines changed: 25 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,15 +71,34 @@ test-dotnet:
7171
@echo "=== Testing .NET code ==="
7272
@cd dotnet && dotnet test test/GitHub.Copilot.SDK.Test.csproj
7373

74-
# Install all dependencies
75-
install:
76-
@echo "=== Installing dependencies ==="
77-
@cd nodejs && npm ci
78-
@cd python && uv pip install -e ".[dev]"
74+
# Install all dependencies across all languages
75+
install: install-go install-python install-nodejs install-dotnet
76+
@echo "✅ All dependencies installed"
77+
78+
# Install Go dependencies and prerequisites for tests
79+
install-go: install-nodejs install-test-harness
80+
@echo "=== Installing Go dependencies ==="
7981
@cd go && go mod download
82+
83+
# Install Python dependencies and prerequisites for tests
84+
install-python: install-nodejs install-test-harness
85+
@echo "=== Installing Python dependencies ==="
86+
@cd python && uv pip install -e ".[dev]"
87+
88+
# Install .NET dependencies and prerequisites for tests
89+
install-dotnet: install-nodejs install-test-harness
90+
@echo "=== Installing .NET dependencies ==="
8091
@cd dotnet && dotnet restore
92+
93+
# Install Node.js dependencies
94+
install-nodejs:
95+
@echo "=== Installing Node.js dependencies ==="
96+
@cd nodejs && npm ci
97+
98+
# Install test harness dependencies (used by E2E tests in all languages)
99+
install-test-harness:
100+
@echo "=== Installing test harness dependencies ==="
81101
@cd test/harness && npm ci --ignore-scripts
82-
@echo "✅ All dependencies installed"
83102

84103
# Run interactive SDK playground
85104
playground:

0 commit comments

Comments
 (0)