Skip to content

feat: Include versions per platform on each registry#44

Merged
kevinszuchet merged 2 commits into
mainfrom
feat/include-versions-per-platform
Aug 11, 2025
Merged

feat: Include versions per platform on each registry#44
kevinszuchet merged 2 commits into
mainfrom
feat/include-versions-per-platform

Conversation

@kevinszuchet

@kevinszuchet kevinszuchet commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Refactored the DB to support version tracking per platform instead of a single global version per registry.

@kevinszuchet
kevinszuchet requested a review from aleortega August 7, 2025 10:32
@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 16802118278

Details

  • 22 of 38 (57.89%) changed or added relevant lines in 6 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.05%) to 55.157%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/logic/handlers/deployment-handler.ts 0 1 0.0%
src/adapters/db.ts 0 4 0.0%
src/migrations/1754552140232_rename-version-to-versions.ts 11 15 73.33%
src/controllers/handlers/post-registry.ts 0 7 0.0%
Files with Coverage Reduction New Missed Lines %
src/adapters/db.ts 1 59.15%
src/controllers/handlers/post-registry.ts 2 2.86%
Totals Coverage Status
Change from base Build 16770485745: 0.05%
Covered Lines: 547
Relevant Lines: 992

💛 - Coveralls

@aleortega aleortega left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

}

// Update version separately
registry = await db.updateRegistryVersion(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

QQ: is this just to simplify upserts?

@kevinszuchet
kevinszuchet merged commit 0ab5770 into main Aug 11, 2025
4 checks passed
@kevinszuchet
kevinszuchet deleted the feat/include-versions-per-platform branch August 11, 2025 07:33
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.

3 participants