Skip to content

Conversation

pamelafox
Copy link
Contributor

Purpose

This PR adds env variables that were missing from the azd pipeline, and updates AGENTS.md with tips for keeping env variables in sync.

Does this introduce a breaking change?

When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.

[ ] Yes
[X] No

Type of change

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Code quality checklist

See CONTRIBUTING.md for more details.

N/A

@pamelafox pamelafox requested a review from Copilot October 3, 2025 21:16
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the Azure DevOps pipeline by adding missing environment variables to the GitHub workflow file and provides documentation for maintaining environment variable consistency. The changes ensure that the azd deployment pipeline has access to all required configuration values.

Key changes:

  • Added comprehensive environment variable mappings to the GitHub workflow
  • Created documentation for managing azd environment variables across different files

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
AGENTS.md New documentation file explaining how to keep azd environment variables in sync across infrastructure files
.github/workflows/azure-dev.yaml Added missing environment variables for OpenAI configurations, deployment settings, and API keys

@pamelafox pamelafox merged commit a7e6cde into main Oct 3, 2025
1 check passed
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.

1 participant