Skip to content

Followup fixes to manifests.txt#91

Merged
Mark-Simulacrum merged 3 commits intomasterfrom
pa-fix-manifests-txt
Jul 3, 2025
Merged

Followup fixes to manifests.txt#91
Mark-Simulacrum merged 3 commits intomasterfrom
pa-fix-manifests-txt

Conversation

@emilyalbini
Copy link
Copy Markdown
Member

This PR contains multiple fixes to #89:

  • It now uses --if-match instead of --if-none-match when the file exists already, as noted in Update manifests.txt from the release process #89 (comment)
  • It adds new manifests at the bottom of the file rather than at the top
  • It handles releases with multiple manifests (for example, one for beta and one for 1.89-beta in the case of beta releases)

I managed to test them locally and everything seems to work fine.

@Mark-Simulacrum Mark-Simulacrum merged commit 27ac1b6 into master Jul 3, 2025
6 checks passed
@Mark-Simulacrum Mark-Simulacrum deleted the pa-fix-manifests-txt branch July 3, 2025 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants