Skip to content

Conversation

@cinnamon-msft
Copy link
Contributor

@cinnamon-msft cinnamon-msft commented Jul 2, 2019

Summary of the Pull Request

Added default values for global and profiles within SettingsSchema.md

References

#1712

PR Checklist

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

| `fontSize` | _Required_ | Integer | `10` | Sets the font size. |
| `guid` | _Required_ | String | | Unique identifier of the profile. Written in registry format: `"{00000000-0000-0000-0000-000000000000}"`. |
| `historySize` | _Required_ | Integer | `9001` | The number of lines above the ones displayed in the window you can scroll back to. |
| `name` | _Required_ | String | `PowerShell Core` | Name of the profile. Displays in the dropdown menu. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no default here really

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should I keep PowerShell guid in globals for defaultProfile?

@Summon528
Copy link
Contributor

How does this close #1753 though

@cinnamon-msft
Copy link
Contributor Author

How does this close #1753 though

Oops, you're right! Fixing issue number now 😊

@miniksa miniksa merged commit eda6547 into master Jul 5, 2019
@miniksa miniksa deleted the cinnamon/settings-schema-update branch July 5, 2019 17:04
mcpiroman pushed a commit to mcpiroman/terminal that referenced this pull request Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation Issue: optional values SettingsSchema should list default value

6 participants