chore: update dependencies and improve workflow configurations #299
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Bump azd version to 1.20.0 in devcontainer.json - Update checkout and setup actions in azure-dev.yml - Add Node.js setup step in terraform-validate.yml - Adjust target framework indentation in Directory.Build.props and CopilotTests.csproj
There was a problem hiding this comment.
Pull Request Overview
This pull request modernizes the development environment and CI/CD workflows by upgrading key tool versions and improving compatibility for self-hosted runners. The changes focus on ensuring the project uses current versions of essential development tools while maintaining stability and security.
Key changes:
- Upgraded Azure Developer CLI from 1.18.1 to 1.20.0 in the development container
- Updated Terraform version from 1.12.2 to 1.13.3 in CI workflows
- Replaced .NET SDK setup action with manual installation script targeting .NET 9.0 for better self-hosted runner compatibility
Reviewed Changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/terraform-validate.yml |
Added Node.js 18.x setup and updated Terraform version to 1.13.3 |
.github/workflows/azure-dev.yml |
Replaced .NET SDK action with manual installation script for .NET 9.0 |
.devcontainer/devcontainer.json |
Updated Azure Developer CLI version to 1.20.0 and improved extension comments |
mattdot
requested changes
Oct 14, 2025
Co-authored-by: Matt Dotson <[email protected]>
Co-authored-by: Matt Dotson <[email protected]>
mattdot
approved these changes
Oct 14, 2025
ianjensenisme
approved these changes
Oct 14, 2025
Contributor
ianjensenisme
left a comment
ianjensenisme
left a comment
There was a problem hiding this comment.
Minor formatting issue but shouldn't block build/merge
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request updates development environment configurations and CI workflows to use newer versions of key tools and improves compatibility for self-hosted runners. The most important changes include upgrading the .NET SDK and Azure Developer CLI versions, updating the Terraform version, and adding Node.js setup in CI.
Development environment upgrades:
.devcontainer/devcontainer.jsonfrom1.18.1to1.20.0for improved features and compatibility.CI workflow improvements:
.github/workflows/azure-dev.ymlto use a manual installation script targeting .NET 9.0, improving compatibility with self-hosted runners and avoiding permission issues..github/workflows/terraform-validate.ymlfrom1.12.2to1.13.3for better support and security.18.x) in the Terraform validation workflow to support Node-based tooling.Minor documentation update:
.devcontainer/devcontainer.jsonfor guidance on adding VSCode extensions.