-
Notifications
You must be signed in to change notification settings - Fork 92
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
winget: auto-update license/copyright urls #562
Conversation
One of our winget manifests currently has hardcoded, outdated values for its LicenseUrl and CopyrightUrl fields. This is because the winget-create tool that we use to automatically generate and publish our manifests cannot change these fields. In light of this, instead of creating and submitting manifests with a single command, update the workflow to do the following: 1. Create the manifests. 2. Update the LicenseUrl and CopyrightUrl fields with the correct version. 3. Submit the manifests. This change also adds the workflow_dispatch trigger to the release-winget workflow for easier testing in the future. Signed-off-by: Lessley Dennington <ledennin@microsoft.com>
Changes since v1:
|
fc8721e
to
e308770
Compare
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.
Looking good. 🤞🏻 that it works on the first try. Otherwise, we can do fixups on top.
$shortenedVersion = $version -replace ".{4}$" | ||
$manifestPath = dir -Path ./manifests -Filter Microsoft.Git.locale.en-US.yaml -Recurse | %{$_.FullName} |
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.
This is some fun script magic. It looks sensible, but we will only be sure once we run it for real.
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.
Welp. I did run it for real yesterday lol. I tried to write to the test repo, but the config failed and submitted a PR to the real repo instead, which I closed immediately. You can check it out here though:
https://github.com/microsoft/winget-pkgs/pull/95353/files
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.
Ah, I see that it wrote https://github.com/microsoft/git/blob/vfs-2.39.10/COPYING
(note .10
instead of .1
).
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.
or, perhaps you had created a v2.39.10.vfs.0.5
tag, based on the rest of the context. Carry on.
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.
Yes, that was one of the experimental tags I created for testing!
One of our winget manifests currently has hardcoded, outdated values for its LicenseUrl and CopyrightUrl fields. This is because the winget-create tool that we use to automatically generate and publish our manifests cannot change these fields. In light of this, instead of creating and submitting manifests with a single command, update the workflow to do the following: 1. Create the manifests. 2. Update the LicenseUrl and CopyrightUrl fields with the correct version. 3. Submit the manifests. This change also adds the workflow_dispatch trigger to the release-winget workflow for easier testing in the future.
One of our winget manifests currently has hardcoded, outdated values for its LicenseUrl and CopyrightUrl fields. This is because the winget-create tool that we use to automatically generate and publish our manifests cannot change these fields. In light of this, instead of creating and submitting manifests with a single command, update the workflow to do the following: 1. Create the manifests. 2. Update the LicenseUrl and CopyrightUrl fields with the correct version. 3. Submit the manifests. This change also adds the workflow_dispatch trigger to the release-winget workflow for easier testing in the future.
One of our winget manifests currently has hardcoded, outdated values for its LicenseUrl and CopyrightUrl fields. This is because the winget-create tool that we use to automatically generate and publish our manifests cannot change these fields. In light of this, instead of creating and submitting manifests with a single command, update the workflow to do the following: 1. Create the manifests. 2. Update the LicenseUrl and CopyrightUrl fields with the correct version. 3. Submit the manifests. This change also adds the workflow_dispatch trigger to the release-winget workflow for easier testing in the future.
One of our winget manifests currently has hardcoded, outdated values for its LicenseUrl and CopyrightUrl fields. This is because the winget-create tool that we use to automatically generate and publish our manifests cannot change these fields.
In light of this, instead of creating and submitting manifests with a single command, update the workflow to do the following:
This change also adds the workflow_dispatch trigger to the release-winget workflow for easier testing in the future.