search

microsoft / vscode-azureappservice

Azure App Service extension for VS Code
https://marketplace.visualstudio.com/items?itemName=ms-azuretools.vscode-azureappservice
MIT License
111 stars 94 forks source link

Remove Csharp tests #2632

Closed motm32 closed 4 months ago

motm32 commented 4 months ago

Doing this since our CodeQL snapshots are failing for csharp and we could either remove the files, add a build task, or create a seperate non-production repo in order to be compliant. After discussion we decided to just remove the tests.

alexweininger commented 4 months ago

Is it possible to verify that this passes CodeQL before merging? If you need to, you can manually run the 1ES pipeline

motm32 commented 4 months ago

Yup I can try doing that!

motm32 commented 4 months ago

Manually ran the 1ES pipeline on this branch and the CodeQL detected languages are now only Javascript and Python so no more warnings from CodeQL about csharp.

nturinski commented 4 months ago

lmao, hooray for compliance actually reducing our test coverage. . . 🎉