Skip to content

Fixes to distribution_manifest_updater.py and publish.yaml - #64

Merged
andrewlukoshko merged 3 commits into
mainfrom
be-more-resilient-to-changes-in-the-github-api-response
Nov 4, 2025
Merged

andrewlukoshko merged 3 commits into
mainfrom
be-more-resilient-to-changes-in-the-github-api-response

Conversation

@yuravk

@yuravk yuravk commented Oct 31, 2025

Copy link
Copy Markdown
Collaborator

tools/distribution_manifest_updater.py:

  • Safely check the API response status.
  • Handle both list and dictionary response formats.
  • Try common dictionary keys (current, latest, default) for version information.
  • Catch exceptions and continues execution with a warning instead of crashing.
  • Correct x64 and ARM64 image file url and checksum file assets.

publish.yaml:

  • Commit and push to almalinux/WSL repository.
  • Correct commit message.
  • Correct PR title and body.
  • Fix the 'Compare distribution manifests' step to complete with correct exit code

…nges in the GitHub API response format

Safely check the API response status.
Handle both list and dictionary response formats.
Try common dictionary keys (current, latest, default) for version information.
Catch exceptions and continues execution with a warning instead of crashing.
Correct x64 and ARM64 image file url and checksum file assets.
Commit and push to almalinux/WSL repository.
Correct commit message.
Correct PR title and body.
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