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

Move as many parameters as possible to Manifest Level #185

Closed
Trenly opened this issue Oct 4, 2021 · 5 comments
Closed

Move as many parameters as possible to Manifest Level #185

Trenly opened this issue Oct 4, 2021 · 5 comments
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work.

Comments

@Trenly
Copy link
Contributor

Trenly commented Oct 4, 2021

Description of the new feature/enhancement

Currently the installer type, scope, upgrade behavior, etc. are always placed at the Installer Level. When these values are the same for all installers, they can be moved to the Manifest Level

Proposed technical implementation details (optional)

Any parameters which are the same across all installers and are valid at both the manifest level and the installer level should be moved to the manifest level during creation or update

@Trenly Trenly added the Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. label Oct 4, 2021
@ghost ghost added the Needs-Triage label Oct 4, 2021
@jedieaston
Copy link
Contributor

I wonder if this should be enforced in a hypothetical schema 2.0. There aren't a ton of circumstances where putting InstallerType or others globally doesn't just turn into a headache later when you want to add a new installer entry.

@Trenly
Copy link
Contributor Author

Trenly commented Oct 4, 2021

I wonder if this should be enforced in a hypothetical schema 2.0. There aren't a ton of circumstances where putting InstallerType or others globally doesn't just turn into a headache later when you want to add a new installer entry.

I think that is an interesting point. I think this is mostly a concern when you manually modify the installer entries for new manifests. If you use a tool or a script like the YamlCreate script, then it should handle it automatically. It certainly is something to consider as to whether or not this would be a good thing

@denelon
Copy link
Contributor

denelon commented Oct 4, 2021

We've had some discussions around this with the feature team. Some of the optimizations like the singleton format and "root" level nodes that can be overridden in a child node were mostly made for humans crafting manifests. As we're getting closer to solid tooling support, these may be deprecated in favor of a tool generated manifest. It would certainly be a breaking change so the 2.0 schema was a good call out by @jedieaston.

@mdanish-kh
Copy link
Contributor

@mdanish-kh
Copy link
Contributor

The fix (PR #409) is available now in WinGet-Create 1.5.1.0

cc @denelon / @Trenly if this can be marked as completed

@Trenly Trenly closed this as completed Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work.
Projects
None yet
Development

No branches or pull requests

4 participants