Skip to content

refactor!: remove legacy assets v1 API#107

Merged
sandovalrr merged 2 commits into
frequenz-floss:v0.x.xfrom
sandovalrr:fix/platformassets-wheel-packaging
Jun 19, 2026
Merged

refactor!: remove legacy assets v1 API#107
sandovalrr merged 2 commits into
frequenz-floss:v0.x.xfrom
sandovalrr:fix/platformassets-wheel-packaging

Conversation

@sandovalrr

@sandovalrr sandovalrr commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • package the generated frequenz.api.platformassets.v1alpha1 runtime modules in wheels
  • remove the legacy frequenz.api.assets.v1 API package
  • keep tests and mypy scoped to frequenz.api.platformassets

Breaking Change

BREAKING CHANGE: The legacy frequenz.api.assets.v1 package was removed. Consumers should use frequenz.api.platformassets.v1alpha1 instead.

@github-actions github-actions Bot added part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) labels Jun 19, 2026
@sandovalrr sandovalrr force-pushed the fix/platformassets-wheel-packaging branch from c721258 to c1f8282 Compare June 19, 2026 11:08
Signed-off-by: Richard Sandoval <rsandovaldev@gmail.com>
Signed-off-by: Richard Sandoval <rsandovaldev@gmail.com>
@sandovalrr sandovalrr force-pushed the fix/platformassets-wheel-packaging branch from c1f8282 to 4b1596b Compare June 19, 2026 11:11

@cwasicki cwasicki left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Not the biggest fan of the renaming to platformassets but apparently happened before the PR already.

@sandovalrr sandovalrr added this pull request to the merge queue Jun 19, 2026
Merged via the queue into frequenz-floss:v0.x.x with commit 24e8f33 Jun 19, 2026
10 checks passed
@sandovalrr sandovalrr deleted the fix/platformassets-wheel-packaging branch June 19, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests part:tooling Affects the development tooling (CI, deployment, dependency management, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants