Commit db4eb52

authored

Fix CI by removing Azure Artifacts Feeds (#4978)

Until it's figured out how we're supposed to use these on GitHub.

1 parent 3a24728 commit db4eb52Copy full SHA for db4eb52

File tree

3 files changed

+12

-34

lines changed

.github/workflows
- ci-test.yml
- npmrc
test/features
- DebugSession.test.ts

3 files changed

+12

-34

lines changed

`‎.github/workflows/ci-test.yml‎`

Lines changed: 10 additions & 18 deletions

Original file line number	Diff line number	Diff line change
`@@ -21,9 +21,6 @@ jobs:`
`21`	`21`	`DOTNET_CLI_TELEMETRY_OPTOUT: true`
`22`	`22`	`DOTNET_GENERATE_ASPNET_CERTIFICATE: false`
`23`	`23`	`DISPLAY: ':99.0'`
`24`		`- defaults:`
`25`		`- run:`
`26`		`- working-directory: vscode-powershell`
`27`	`24`	`steps:`
`28`	`25`	`- name: Checkout PowerShellEditorServices`
`29`	`26`	`uses: actions/checkout@v4`
`@@ -40,24 +37,20 @@ jobs:`
`40`	`37`	`uses: actions/setup-dotnet@v4`
`41`	`38`	`with:`
`42`	`39`	`cache: true`
`43`		`- cache-dependency-path: 'PowerShellEditorServices/**/packages.lock.json'`
	`40`	`+ cache-dependency-path: PowerShellEditorServices/**/packages.lock.json`
`44`	`41`	`global-json-file: PowerShellEditorServices/global.json`
`45`		`- source-url: https://pkgs.dev.azure.com/mscodehub/PowerShellCore/_packaging/PowerShellCore_PublicPackages/nuget/v3/index.json`
`46`		`- config-file: PowerShellEditorServices/NuGet.config`
`47`		`- env:`
`48`		`- NUGET_AUTH_TOKEN: ${{ secrets.AZURE_NUGET_TOKEN }}`
`49`		`-`
`50`		`- - name: Install PSResources`
`51`		`- shell: pwsh`
`52`		`- run: ./tools/installPSResources.ps1`
`53`	`42`
`54`		`- - name: Deploy generated NuGet configuration`
	`43`	`+ - name: Remove Azure Artifact Feed configurations`
`55`	`44`	`shell: pwsh`
`56`		`- run: Copy-Item ../../nuget.config ../PowerShellEditorServices/NuGet.config`
	`45`	`+ run: \|`
	`46`	`+ Remove-Item -Force .npmrc`
	`47`	`+ Remove-Item test/mocks/BinaryModule/NuGet.Config`
	`48`	`+ Remove-Item ../PowerShellEditorServices/NuGet.Config`
	`49`	`+ working-directory: vscode-powershell`
`57`	`50`
`58`		`- - name: Deploy NPM configuration`
	`51`	`+ - name: Install PSResources`
`59`	`52`	`shell: pwsh`
`60`		`- run: Copy-Item .github/workflows/npmrc .npmrc`
	`53`	`+ run: ./vscode-powershell/tools/installPSResources.ps1`
`61`	`54`
`62`	`55`	`- uses: actions/setup-node@v4`
`63`	`56`	`with:`
`@@ -72,8 +65,7 @@ jobs:`
`72`	`65`	`- name: Build, test and package`
`73`	`66`	`shell: pwsh`
`74`	`67`	`run: Invoke-Build -Configuration Release`
`75`		`- env:`
`76`		`- NPM_PASSWORD: ${{ secrets.AZURE_NPM_PASSWORD_BASE64 }}`
	`68`	`+ working-directory: vscode-powershell`
`77`	`69`
`78`	`70`	`- name: Upload build artifacts`
`79`	`71`	`uses: actions/upload-artifact@v4`

`‎.github/workflows/npmrc‎`

Lines changed: 0 additions & 13 deletions

This file was deleted.

`‎test/features/DebugSession.test.ts‎`

Lines changed: 2 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -486,9 +486,8 @@ describe("DebugSessionFeature E2E", function() {`
`486`	`486`	`let binaryModulePath: Uri;`
`487`	`487`
`488`	`488`	`before(async function binarySetup() {`
`489`		`- if (process.env.TF_BUILD) {`
`490`		`- // The binary modules tests won't work in the release pipeline`
`491`		`- // due to dependency requirements.`
	`489`	`+ if (!extensions.getExtension("ms-dotnettools.csharp")) {`
	`490`	`+ // These tests require that extension to be installed in the test environment.`
`492`	`491`	`this.skip();`
`493`	`492`	`}`
`494`	`493`	`binaryModulePath = Uri.joinPath(workspace.workspaceFolders![0].uri, "BinaryModule");`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit db4eb52

File tree

3 files changed

3 files changed

`‎.github/workflows/ci-test.yml‎`

`‎.github/workflows/npmrc‎`

`‎test/features/DebugSession.test.ts‎`

0 commit comments