Skip to content

Support versioned FOSSBilling zip assets#121

Merged
admdly merged 2 commits into
mainfrom
fix/release-naming
May 28, 2026
Merged

Support versioned FOSSBilling zip assets#121
admdly merged 2 commits into
mainfrom
fix/release-naming

Conversation

@admdly
Copy link
Copy Markdown
Contributor

@admdly admdly commented May 28, 2026

No description provided.

@admdly admdly self-assigned this May 28, 2026
Copilot AI review requested due to automatic review settings May 28, 2026 21:49
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 28, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
api cfe8449 Commit Preview URL

Branch Preview URL
May 28 2026, 09:56 PM

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support in the Versions API v1 for GitHub releases whose ZIP asset is named with the release version (e.g., FOSSBilling-0.8.0.zip) instead of only FOSSBilling.zip, ensuring those releases are included in the API response.

Changes:

  • Updated release parsing to accept either FOSSBilling.zip or FOSSBilling-${tag}.zip as the downloadable asset.
  • Added a regression test covering a release that only contains a versioned ZIP asset name.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
test/services/versions/v1/index.test.ts Adds a test verifying versioned ZIP asset names are recognized and surfaced in the API response.
src/services/versions/v1/index.ts Introduces a helper to locate the correct ZIP asset (unversioned or versioned) when building release metadata.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@admdly admdly merged commit 33344f3 into main May 28, 2026
8 checks passed
@admdly admdly deleted the fix/release-naming branch May 28, 2026 22:00
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