Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

schema: add reloadEnvironmentVariables to newTerminalArgs #17696

Merged
merged 3 commits into from
Aug 14, 2024

Conversation

carlos-zamora
Copy link
Member

Updates the schema to include reloadEnvironmentVariables for New Terminal Args.

Other properties that aren't included:

  • __content: we're explicitly not including this in the schema since it's only used internally. In fact, we even throw an error if it is defined (sample code)!
  • sessionId: there isn't really a legit use for this one, so we'll leave it undocumented and as an implementation detail.

Closes #17469

@microsoft-github-policy-service microsoft-github-policy-service bot added Issue-Docs It's a documentation issue that really should be on MicrosoftDocs/Console-Docs Area-Schema Things that have to do with the json schema. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. labels Aug 9, 2024
@DHowett DHowett changed the title Add reloadEnvironmentVariables as a property on newTerminalArgs to schema schema: add reloadEnvironmentVariables to newTerminalArgs Aug 9, 2024
doc/cascadia/profiles.schema.json Outdated Show resolved Hide resolved
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Aug 13, 2024
Co-authored-by: Dustin L. Howett <[email protected]>
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Aug 13, 2024
@DHowett DHowett merged commit 1511d2c into main Aug 14, 2024
12 checks passed
@DHowett DHowett deleted the dev/cazamor/schema/new-term-args branch August 14, 2024 20:16
DHowett pushed a commit that referenced this pull request Aug 20, 2024
(cherry picked from commit 1511d2c)
Service-Card-Id: 93522518
Service-Version: 1.20
DHowett pushed a commit that referenced this pull request Aug 20, 2024
(cherry picked from commit 1511d2c)
Service-Card-Id: 93522519
Service-Version: 1.21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Schema Things that have to do with the json schema. Issue-Docs It's a documentation issue that really should be on MicrosoftDocs/Console-Docs Priority-3 A description (P3) Product-Terminal The new Windows Terminal.
Projects
Status: Cherry Picked
Development

Successfully merging this pull request may close these issues.

profiles.schema.json: NewTerminalArgs section is outdated
3 participants