Skip to content
This repository was archived by the owner on Nov 15, 2025. It is now read-only.

Conversation

Copy link

Copilot AI commented Oct 21, 2025

Coding agent has begun work on Rename LLM_MODEL to ... and will update this pull request as work progresses.

Original prompt

Rename LLM_MODEL to AZURE_DEPLOYMENT_NAME for Azure OpenAI clarity

This PR renames the environment variable from LLM_MODEL to AZURE_DEPLOYMENT_NAME for better clarity and consistency with Azure OpenAI deployments.

Changes Made

  • Documentation: Updated README.md to reflect the new environment variable name in Azure target examples
  • Configuration: Modified targets.yaml files to use AZURE_DEPLOYMENT_NAME instead of LLM_MODEL
  • Templates: Updated .env.template with the new variable name
  • Code Changes:
    • Updated cli.py to properly resolve the model setting from target configurations
    • Modified azure_deployment_diag.py to use the new environment variable
  • Version Bump: Incremented version from 0.1.9 to 0.1.10 in pyproject.toml

Motivation

The previous naming LLM_MODEL was generic and could be confusing. AZURE_DEPLOYMENT_NAME is more specific and clearly indicates it's for Azure OpenAI deployment names, improving developer experience and reducing configuration errors.

Testing

  • Verified that the CLI properly resolves environment variables from target settings
  • Confirmed diagnostic script uses the correct variable
  • Updated all example configurations to use the new naming

Created from VS Code via the GitHub Pull Request extension.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

christso and others added 3 commits October 19, 2025 15:10
…or Azure OpenAI deployments, update documentation and configuration files accordingly, and bump version to 0.1.10
Copilot AI requested a review from christso October 21, 2025 05:05
Base automatically changed from rename-openai to main October 21, 2025 05:05
@christso christso closed this Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants