-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Create ManifestSpecv1.0.md #816
Conversation
Manifest Schema v1.0
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
PackageVersion: "1.6.10571.0" | ||
PackageLocale: "fr-FR" | ||
Publisher: "Microsoft" | ||
ShortDescription: "Le nouveau terminal Windows, une expérience de ligne de commande à onglets pour Windows." |
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.
Easiest way to ignore this is to add something like:
^ShortDescription: "Le nouveau.*"$
to patterns.txt
(preferably w/ a comment referencing this line).
Adjustments to language to provide greater clarity.
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
- Architecture: "x64" | ||
InstallerUrl: "https://github.com/microsoft/terminal/releases/download/v1.6.10571.0/Microsoft.WindowsTerminal_1.6.10571.0_8wekyb3d8bbwe.msixbundle" | ||
InstallerSha256: 092aa89b1881e058d31b1a8d88f31bb298b5810afbba25c5cb341cfa4904d843 | ||
SignatureSha256: e53f48473621390c8243ada6345826af7c713cf1f4bbbf0d030599d1e4c175ee |
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.
Is InstallerType no longer required?
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.
Great catch! I've been working on this one so much I missed it.
Installer Type
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
1 similar comment
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
Misspellings found, please review:
To accept these changes, run the following commands from this repository on this branch
|
I wonder if we could have field for windows shim like wt.exe of windows terminal, or wsl.exe, unbuntu.exe of wsl distro? |
That should be going in the command field, I think. |
@quangkieu, yes the "commands" key is intended to identify the command one would execute to run the program/package. It's currently just an informational field, but that is the intent. |
I wonder how about winget generate one, for example the unpacked zip install, or git.exe so we only need folder windowsapp for those shim but the app could install, unpack in correct folder. We also have a flag if user want to disable the generated shim? |
Our plan for supporting .zip and standalone .exe as a package is a bit different. We're looking at having a path where they will be placed on a drive, and an entry in the PATH environment variable. We will also add an entry in Add/Remove Programs so they can be removed and upgraded as other packages are. This Add/Remove Programs integration would require a more recent version of Windows, but the package manager would still be able to handle the scenarios for uninstall and upgrade. |
My only concern is the amount of entries on PATH variable. And if other microsoft team, product already has shim feature, I would want to suggest to improve and promote it to be official solution instead just patch work in those products only |
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.
Looks great.
Manifest Schema v1.0
Microsoft Reviewers: Open in CodeFlow