Skip to content

Commit 1be624a

Browse files
committed
Install language server depenedencies as part of npm i
1 parent f6ef6a3 commit 1be624a

11 files changed

Lines changed: 21436 additions & 21403 deletions

File tree

.devcontainer/devcontainer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,5 +56,5 @@
5656
]
5757
}
5858
},
59-
"postCreateCommand": "npm ci && node run installDependencies"
59+
"postCreateCommand": "npm ci"
6060
}

.github/workflows/copilot-setup-steps.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,3 @@ jobs:
4343
# Install vsce (needed for packaging tasks)
4444
- name: Install vsce
4545
run: npm install -g vsce
46-
47-
# Install server dependencies (needed for integration tests and running)
48-
- name: Install dependencies
49-
run: npm run installDependencies
50-
51-

CONTRIBUTING.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@ Follow these steps to build, run, and test the repository:
3939
If you have the ability to run powershell, you can invoke "init.ps1" from the root of the repo. If not, the following steps will get build going for you as well:
4040

4141
1. Run `npm ci` - This command installs the project dependencies.
42-
2. Run `npm run installDependencies` - This command downloads the various dependencies as specified by the version in the [package.json](package.json) file.
4342

4443
**Note**: Authentication with `ado-npm-auth` is only required when adding new packages to the feeds. For regular development with existing dependencies, authentication is not necessary. See the [Updating NPM packages](#updating-npm-packages) section for details.
4544

azure-pipelines/test-prereqs.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
steps:
22

3-
# We need to download the roslyn language server for the current platform to run integration tests against it.
4-
- script: npm run installDependencies
5-
displayName: 'Install Roslyn Language Server'
6-
73
- script: /usr/bin/Xvfb :99 -screen 0 1024x768x24 > /dev/null 2>&1 &
84
displayName: 'Activate screen emulation'
95
condition: eq(variables['Agent.OS'], 'Linux')

init.ps1

Lines changed: 4 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -18,37 +18,22 @@ function Run-Command($command, $arguments, $errorMsg) {
1818
}
1919
}
2020

21-
2221
Push-Location $PSScriptRoot
2322

2423
try {
25-
Write-Host "`n[1/5] Installing ado-npm-auth globally..." -ForegroundColor Cyan
24+
Write-Host "`n[1/3] Installing ado-npm-auth globally..." -ForegroundColor Cyan
2625
Run-Command "npm" @("install", "-g", "ado-npm-auth") "Failed to install ado-npm-auth."
2726

28-
Write-Host "`n[2/5] Authenticating with Azure DevOps..." -ForegroundColor Cyan
27+
Write-Host "`n[2/3] Authenticating with Azure DevOps..." -ForegroundColor Cyan
2928
if (Test-Path ".npmrc") {
3029
Run-Command "ado-npm-auth" @("-c", ".npmrc") "Authentication failed."
3130
} else {
3231
Write-Host ".npmrc file not found in the current directory." -ForegroundColor Red
3332
throw ".npmrc file not found in the current directory."
3433
}
3534

36-
Write-Host "`n[1/4] Installing ado-npm-auth globally..." -ForegroundColor Cyan
37-
Run-Command "npm" @("install", "-g", "ado-npm-auth") "Failed to install ado-npm-auth."
38-
39-
Write-Host "`n[2/4] Authenticating with Azure DevOps..." -ForegroundColor Cyan
40-
if (Test-Path ".npmrc") {
41-
Run-Command "ado-npm-auth" @("-c", ".npmrc") "Authentication failed."
42-
} else {
43-
Write-Host ".npmrc file not found in the current directory." -ForegroundColor Red
44-
throw ".npmrc file not found in the current directory."
45-
}
46-
47-
Write-Host "`n[3/4] Installing project dependencies..." -ForegroundColor Cyan
48-
Run-Command "npm" @("install") "Failed to install project dependencies."
49-
50-
Write-Host "`n[4/4] Running installDependencies..." -ForegroundColor Cyan
51-
Run-Command "node" @("run", "installDependencies") "Failed to run 'installDependencies'."
35+
Write-Host "`n[3/3] Installing project dependencies..." -ForegroundColor Cyan
36+
Run-Command "npm" @("ci") "Failed to install project dependencies."
5237

5338
Write-Host "`n✅ Setup complete." -ForegroundColor Green
5439
}

0 commit comments

Comments
 (0)