Skip to content

Change Claude Code environment variables format for vscode extension#749

Open
nmellal wants to merge 1 commit into
MicrosoftDocs:mainfrom
nmellal:patch-1
Open

Change Claude Code environment variables format for vscode extension#749
nmellal wants to merge 1 commit into
MicrosoftDocs:mainfrom
nmellal:patch-1

Conversation

@nmellal
Copy link
Copy Markdown

@nmellal nmellal commented May 20, 2026

Updated the configuration format for Claude Code environment variables from an object to an array for vscode extension. The actual format doesn't work

@nmellal
Change Claude Code environment variables format for vscode extension
a5396d5 
Updated the configuration format for Claude Code environment variables from an object to an array for vscode extension.
The actual format doesn't work
@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented May 20, 2026

@nmellal : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented May 20, 2026

Learn Build status updates of commit a5396d5:

✅ Validation status: passed

File Status Preview URL Details
articles/foundry/foundry-models/how-to/configure-claude-code.md ✅Succeeded

For more details, please refer to the build report.

@v-regandowner
Copy link
Copy Markdown
Contributor

v-regandowner commented May 20, 2026

@msakande

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged C+L Pull Request Review Team label label May 20, 2026
@v-regandowner v-regandowner requested a review from Copilot May 20, 2026 14:18
Copilot AI reviewed May 20, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions.
Get Help
Docs Support Teams Channel
Resolve Merge Conflict

@github-actions github-actions Bot added the inactive This PR is inactive for more than 14 days label Jun 3, 2026
@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Jun 3, 2026

@nmellal : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@msakande msakande Awaiting requested review from msakande

@AmarBadal AmarBadal Awaiting requested review from AmarBadal

Assignees

@msakande msakande

Labels

aq-pr-triaged C+L Pull Request Review Team label Change sent to author do-not-merge foundry-models/subsvc inactive This PR is inactive for more than 14 days microsoft-foundry/svc

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nmellal @v-regandowner @msakande