Add Copilot instructions for repository workflow and testing#2537
Add Copilot instructions for repository workflow and testing#2537
Conversation
Co-authored-by: nagilson <23152278+nagilson@users.noreply.github.com>
Co-authored-by: nagilson <23152278+nagilson@users.noreply.github.com>
MiYanni
left a comment
There was a problem hiding this comment.
Only question... did you test if Copilot uses these instructions properly? (I don't know how it handles these files, so I'm just asking in general.)
|
It does, although I think it can be truncated, and we can also teach it how to login through vsts, and how to update dependency versions and such. |
|
Please reduce the duplicated information in this instructions file.
Write-Host "Starting comprehensive update and build process..." -ForegroundColor Yellow
# Root directory updates
Write-Host "Updating root directory..." -ForegroundColor Cyan
npm update
yarn install
yarn upgrade
# vscode-dotnet-runtime-library
Write-Host "Updating vscode-dotnet-runtime-library..." -ForegroundColor Cyan
Set-Location .\vscode-dotnet-runtime-library\
npm update
yarn install
yarn upgrade
# vscode-dotnet-runtime-extension
Write-Host "Updating vscode-dotnet-runtime-extension..." -ForegroundColor Cyan
Set-Location ..\vscode-dotnet-runtime-extension\
npm update
yarn install
yarn upgrade
npm version patch
# Back to vscode-dotnet-runtime-library
Write-Host "Final update to vscode-dotnet-runtime-library..." -ForegroundColor Cyan
Set-Location ..\vscode-dotnet-runtime-library\
npm update
yarn upgrade
# sample
Write-Host "Updating sample..." -ForegroundColor Cyan
Set-Location ..\sample\
npm update
yarn upgrade
npm version patch
# vscode-dotnet-sdk-extension
Write-Host "Updating vscode-dotnet-sdk-extension..." -ForegroundColor Cyan
Set-Location ..\vscode-dotnet-sdk-extension\
npm update
yarn upgrade
# Back to root and build
Write-Host "Running build..." -ForegroundColor Cyan
Set-Location ..
.\build.cmd
Write-Host "Update and build process completed!" -ForegroundColor Green
```
Note: when running build or npm ci / npm install, you may hit `401` or `403` or `unauth` like errors. This is because you need to login:
`npm install -g vsts-npm-auth` (only available on windows)
`vsts-npm-auth -config .npmrc -force`
Sometimes, when you try to run `yarn upgrade` or `vsce` to package the extension vsix, it will fail because azure devops is missing the package. You need to make a test folder and install the specific package that yarn couldn't find using npm install package@version, and then you can delete that test folder and rerun the yarn check. It may require several pushes, but that npm install is basically updating the upstream azure devops with the subdependencies that yarn is trying to resolve. |
…ructions Co-authored-by: nagilson <23152278+nagilson@users.noreply.github.com>
Yes! GitHub Copilot automatically reads and applies instructions from
Done in commit 5144e02. Reduced the file from 351 to 227 lines (35% reduction) by removing duplication while adding the dependency update workflow and version bumping guidance as requested. |
Co-authored-by: nagilson <23152278+nagilson@users.noreply.github.com>
Co-authored-by: nagilson <23152278+nagilson@users.noreply.github.com>
|
@copilot when bumping the extension version, please also update the changelog file. |
Co-authored-by: nagilson <23152278+nagilson@users.noreply.github.com>
Copilot Instructions Setup Plan
.github/copilot-instructions.mdfile with comprehensive repository guidanceOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.