-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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
Updated schema with newTabPosition setting #13469
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Hello @zadjii-msft! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
@zadjii-msft automerge doesn't do anything for doc-only changes haha. I'll merge this manually right now. |
} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This appears to be one-too-many closing braces. VSCode is complaining:
Unable to parse content from 'https://raw.githubusercontent.com/microsoft/terminal/2f58711ee6b27064d11dec84010353e144f170f0/doc/cascadia/profiles.schema.json': Parse error at offset 81576.
when used as a $schema
value, but does not complain about the previous commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this happed with the merge from main
Renames `newTabPosition` strings and enums. Also fixes the schema, noted in #13469 (comment) Closes #13597
🎉 Handy links: |
Summary of the Pull Request
Updates schema with newTabPosition global setting, added in #13421
PR Checklist
newTabPosition
MicrosoftDocs/terminal#559Detailed Description of the Pull Request / Additional comments
#13421 got merged, so I'm adding this one