[PR #5304] [MERGED] doc: clean up the schema #26247

Open
opened 2026-01-31 09:14:54 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5304
Author: @carlos-zamora
Created: 4/9/2020
Status: Merged
Merged: 4/28/2020
Merged by: @DHowett-MSFT

Base: masterHead: dev/cazamor/schema/nullable-colors


📝 Commits (8)

📊 Changes

1 file changed (+39 additions, -116 deletions)

View changed files

📝 doc/cascadia/profiles.schema.json (+39 -116)

📄 Description

Summary of the Pull Request

This PR performs a number of miscellaneous clean-up tasks on our schema:

  • allow null for some settings
  • update some default values to be more useful
  • consistently use " instead of '
  • remove colorTable
  • provide missing description for backgroundImageOpacity
  • update cursorShape to have options match order displayed in vs code
  • if a setting has multiple options, provide a list of those options with a short description on each
    • this one I only did a few times
  • remove a little bit of ambiguity on some descriptions

PR Checklist

Closes #5279

Detailed Description of the Pull Request / Additional comments

Validation was performed for...

  • Global Settings
  • Profile Settings
  • Keybinding Actions

Validation Steps Performed

I linked VS Code to the my version of the schema. Defined all settings from scratch in settings.json. Anytime I found something off, I updated the schema and saw if it looks right.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/5304 **Author:** [@carlos-zamora](https://github.com/carlos-zamora) **Created:** 4/9/2020 **Status:** ✅ Merged **Merged:** 4/28/2020 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/cazamor/schema/nullable-colors` --- ### 📝 Commits (8) - [`708de5e`](https://github.com/microsoft/terminal/commit/708de5e0f69846c438087accdf5d5f14018de412) draft: clean up schema - [`94db358`](https://github.com/microsoft/terminal/commit/94db358948deb45c65da21b13859829feedf3f5b) address a few of Dustin's comments - [`db517d8`](https://github.com/microsoft/terminal/commit/db517d87522038de4c6ace99325b08bd10e38912) added a few more fixes from Dustin - [`de49a6e`](https://github.com/microsoft/terminal/commit/de49a6e7f2452572ae09c3b0b2712449b001347a) tweak keybinding schema - [`84ebd21`](https://github.com/microsoft/terminal/commit/84ebd21ff5adf7bbc25595d0e53d1a1a9a37fd95) a few PR changes - [`0217cad`](https://github.com/microsoft/terminal/commit/0217cadf0e03106fe374209e8c39d9066ca6ef7b) remove connectionType - [`00b233a`](https://github.com/microsoft/terminal/commit/00b233ad501afbb20f81bfc4297d300f86ea6eed) let's ship this - [`5aec1b9`](https://github.com/microsoft/terminal/commit/5aec1b93945920c1f2ad00503bc0741a20bf09c6) merge master ### 📊 Changes **1 file changed** (+39 additions, -116 deletions) <details> <summary>View changed files</summary> 📝 `doc/cascadia/profiles.schema.json` (+39 -116) </details> ### 📄 Description ## Summary of the Pull Request This PR performs a number of miscellaneous clean-up tasks on our schema: - [X] allow null for some settings - [X] update some default values to be more useful - [X] consistently use " instead of ' - [X] remove colorTable - [X] provide missing description for `backgroundImageOpacity` - [X] update cursorShape to have options match order displayed in vs code - [X] if a setting has multiple options, provide a list of those options with a short description on each - this one I only did a few times - [X] remove a little bit of ambiguity on some descriptions ## PR Checklist Closes #5279 ## Detailed Description of the Pull Request / Additional comments Validation was performed for... - [X] Global Settings - [X] Profile Settings - [x] Keybinding Actions ## Validation Steps Performed I linked VS Code to the my version of the schema. Defined all settings from scratch in settings.json. Anytime I found something off, I updated the schema and saw if it looks right. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:14:54 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#26247