forked from microsoft/winget-pkgs
-
Notifications
You must be signed in to change notification settings - Fork 1
Refactoring of the script to update the manifests in-place where applicable #37
Copy link
Copy link
Open
Description
Upon further review; This should also include a refactoring of the script to update the manifests in-place where applicable, instead of always creating new ones.
Logic Flow:
------------------------------
During read previous flow
------------------------------
If old singleton manifest exists
> Create new empty multi-manifest
> Copy existing fields from singleton into the appropriate multi-manifest file
If old multi-manifest exists
> Set new installer manifest equal to existing installer manifest
> Set new locale manifest equal to existing locale manifest
> Set new version manifest equal to old locale manifest
If no manifests exist
> Create new empty manifests
------------------------------
During data entry flow
------------------------------
For any new required entries
> Set the appropriate Manifest['Key'] value directly instead of using script variables
For any optional values
> If the value entered is null or empty, do not update the field
> If the value entered is not null and is valid, update the value in place
------------------------------
During file creation flow
------------------------------
Directly use the new manifests which were updated in-place
Re-Sort the keys to ensure accuracy
Originally posted by @Trenly in #30 (comment)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels