[PR #11360] [MERGED] Fix compatibility issues with profiles.schema.json #28518

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/11360
Author: @lhecker
Created: 9/28/2021
Status: Merged
Merged: 9/29/2021
Merged by: @zadjii-msft

Base: mainHead: dev/lhecker/schema-correctness


📝 Commits (2)

  • 3b8b964 Fix $schema & Rename definitions to official $defs
  • caba633 Reformat profiles.schema.json with VS Code

📊 Changes

1 file changed (+569 additions, -218 deletions)

View changed files

📝 doc/cascadia/profiles.schema.json (+569 -218)

📄 Description

This fixes two issues with profiles.schema.json:

  • The $schema should not end in a #
  • $defs is the official reserved keyword for schema re-use

See: http://json-schema.org/draft/2020-12/json-schema-core.html

PR Checklist

  • I work here
  • Tests added/passed
  • Schema updated

Validation Steps Performed

The previous schema didn't pass https://jschon.dev/, the new schema does.


🔄 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/11360 **Author:** [@lhecker](https://github.com/lhecker) **Created:** 9/28/2021 **Status:** ✅ Merged **Merged:** 9/29/2021 **Merged by:** [@zadjii-msft](https://github.com/zadjii-msft) **Base:** `main` ← **Head:** `dev/lhecker/schema-correctness` --- ### 📝 Commits (2) - [`3b8b964`](https://github.com/microsoft/terminal/commit/3b8b964f6015d37f8ba3e25575d99ae275a79984) Fix $schema & Rename definitions to official $defs - [`caba633`](https://github.com/microsoft/terminal/commit/caba6338011dfbc4b19955462ab09592ba15d5fb) Reformat profiles.schema.json with VS Code ### 📊 Changes **1 file changed** (+569 additions, -218 deletions) <details> <summary>View changed files</summary> 📝 `doc/cascadia/profiles.schema.json` (+569 -218) </details> ### 📄 Description This fixes two issues with profiles.schema.json: * The `$schema` should not end in a `#` * `$defs` is the official reserved keyword for schema re-use See: http://json-schema.org/draft/2020-12/json-schema-core.html ## PR Checklist * [x] I work here * [x] Tests added/passed * [x] Schema updated ## Validation Steps Performed The previous schema didn't pass https://jschon.dev/, the new schema does. --- <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:29:02 +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#28518