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

WIP: Added support for Windows Terminal v0.11.1121.0 or newer #56

Draft
wants to merge 1 commit into
base: preview
Choose a base branch
from
Draft

WIP: Added support for Windows Terminal v0.11.1121.0 or newer #56

wants to merge 1 commit into from

Conversation

jkrgr0
Copy link

@jkrgr0 jkrgr0 commented May 7, 2020

In version 0.11.1121.0 of Windows Terminal the profiles.json has been renamed to settings.json (See microsoft/terminal Release notes).

To support this change and also to support older releases, I've added a internal function which determines the configuration file path for Windows Terminal.

@jkrgr0
Copy link
Author

jkrgr0 commented May 7, 2020

I've determined, that also the structure of the profile list has slightly changed.
I will adding support for this the next days and keep pushing the changes to this branch.
So the pull request is WIP

@jkrgr0 jkrgr0 changed the title Added support for Windows Terminal v0.11.1121.0 or newer WIP: Added support for Windows Terminal v0.11.1121.0 or newer May 7, 2020
@jkrgr0 jkrgr0 marked this pull request as draft May 7, 2020 19:56
@JustinGrote
Copy link
Collaborator

@wav3 yes there have been many changes, we are working on a complete refactor for 2.0:
https://github.com/JustinGrote/MSTerminalSettings/tree/refactor/QuickType

If you get this fixed for the existing implementation, it can be maintained for the 1.0 branch, otherwise you can try the 2.0 preview

@jkrgr0
Copy link
Author

jkrgr0 commented May 7, 2020

To be honest, I didn't even notice the other branches..

On the first view it does not seems to be too complex to fix the settings json for the existing implementation. I will dig deeper in this on the weekend.

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.

2 participants