Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Sep 12, 2025

This PR modernizes the azure.yaml configuration by removing the pipeline: variables: section, which is no longer needed as of Azure Developer CLI version 1.17.0. The Azure Developer CLI now auto-detects the needed variables from main.parameters.json.

Changes Made

  1. Removed the entire pipeline: block containing 14 variable definitions that are now redundant
  2. Updated requiredVersions.azd from >= 1.10.0 to >= 1.17.0 to enforce the minimum version that supports auto-detection
  3. Bumped metadata.template version from 0.2.0 to 0.3.0 to indicate this modernization

Before

requiredVersions:
    azd: ">= 1.10.0"
# ... services configuration ...
pipeline:
  variables:
    - AZURE_OPENAI_CHAT_DEPLOYMENT
    - AZURE_OPENAI_CHAT_MODEL
    # ... 12 more variables ...

After

requiredVersions:
    azd: ">= 1.17.0"
# ... services configuration (no pipeline section) ...

All the removed variables are properly defined in infra/main.parameters.json with the ${VARIABLE_NAME=default} syntax, ensuring azd >= 1.17.0 can auto-detect them. The CI workflows continue to work as they define these same variables as GitHub variables/secrets.

Benefits

  • Simplified configuration - Variables are now inferred automatically
  • Reduced duplication - Single source of truth in main.parameters.json
  • Reduced drift risk - No need to maintain variables in multiple places
  • Aligned with best practices - Uses modern azd capabilities

Fixes #329.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…ion requirement

Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Remove the pipeline: variables section from azure.yaml Remove pipeline variables section from azure.yaml and update azd version requirement Sep 12, 2025
@Copilot Copilot AI requested a review from pamelafox September 12, 2025 16:48
Copilot finished work on behalf of pamelafox September 12, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove the pipeline: variables section from azure.yaml
2 participants