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

Finish xtask manifest subcommand #714

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Conversation

j-lanson
Copy link
Collaborator

@j-lanson j-lanson commented Dec 11, 2024

#693 must be merged first.

Resolves #689 .

Using the DownloadManifestEntry vecs parsed from the GitHub release API, diffs with the local download manifest entries of a plugin to determine which need to be appended to the file, and does the appending.

Only deficiency is that I was unable to determine how to make the KdlNodes print with indent=2 and insert underscores in large numbers.

@j-lanson j-lanson added type: enhancement New feature or request product: project Relates to the project itself labels Dec 11, 2024
@j-lanson j-lanson added this to the 3.9.0 milestone Dec 11, 2024
@j-lanson j-lanson self-assigned this Dec 11, 2024
@j-lanson j-lanson force-pushed the jlanson/xtask-manifest-diff-site branch 2 times, most recently from 933488e to 48c9ab7 Compare December 16, 2024 19:53
alilleybrinker
alilleybrinker previously approved these changes Dec 16, 2024
@j-lanson j-lanson dismissed alilleybrinker’s stale review December 16, 2024 21:31

The merge-base changed after approval.

@j-lanson j-lanson force-pushed the jlanson/xtask-manifest-diff-site branch from 48c9ab7 to dbeb1c0 Compare December 16, 2024 21:31
@j-lanson j-lanson merged commit a02ceea into main Dec 17, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product: project Relates to the project itself type: enhancement New feature or request
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

🛤️ Tracking: xtask to update plugin download manifests on release
2 participants