Skip to content

Require quantities 1.0.2 for marshmallow 4 - #1454

Closed
Marenz wants to merge 1 commit into
frequenz-floss:v1.x.xfrom
Marenz:chore/quantities-1.0.2-floor
Closed

Require quantities 1.0.2 for marshmallow 4#1454
Marenz wants to merge 1 commit into
frequenz-floss:v1.x.xfrom
Marenz:chore/quantities-1.0.2-floor

Conversation

@Marenz

@Marenz Marenz commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Require the marshmallow 4-compatible quantities release.

Update configuration validation tests for marshmallow 4.

@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 Sep 7, 2026
shsms
shsms previously approved these changes Sep 7, 2026
@github-project-automation github-project-automation Bot moved this from To do to Review approved in Python SDK Roadmap Sep 7, 2026
Comment thread tests/config/test_manager.py Outdated
name: str = dataclasses.field(metadata={"validate": lambda s: s.startswith("test")})
name: str = dataclasses.field(
metadata={
"validate": marshmallow.validate.Regexp("^test", error="Invalid value.")

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.

Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
@llucax

llucax commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

I think this can be closed now.

@Marenz Marenz closed this Sep 9, 2026
@github-project-automation github-project-automation Bot moved this from Review approved to Done in Python SDK Roadmap Sep 9, 2026
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

Development

Successfully merging this pull request may close these issues.

3 participants