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

Rename LLM_MODEL to AZURE_DEPLOYMENT_NAME#14

Merged
christso merged 2 commits into
mainfrom
rename-openai
Oct 21, 2025
Merged

Rename LLM_MODEL to AZURE_DEPLOYMENT_NAME#14
christso merged 2 commits into
mainfrom
rename-openai

Conversation

@christso

Copy link
Copy Markdown
Contributor

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 \ argets.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 \�zure_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

…or Azure OpenAI deployments, update documentation and configuration files accordingly, and bump version to 0.1.10
@christso christso merged commit 15d57d9 into main Oct 21, 2025
@christso christso deleted the rename-openai branch October 21, 2025 05:05
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.

1 participant