Skip to content

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Nov 12, 2024

These quantities were mistakenly excluded from marshmallow fields, but they should be provided too.

Also make sure a ValidationError is raised if there is any issue creating the quantities.

For quantity marshmallow fields, we convert any `Exception` to a
`ValidationError`, so the error reporting includes the *offending* field
appropriately and it is considered a regular validation error.

Signed-off-by: Leandro Lucarella <[email protected]>
These quantities were mistakenly excluded from marshmallow fields, but
they should be provided too.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax llucax requested a review from a team as a code owner November 12, 2024 09:22
@github-actions github-actions bot added part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests part:code Affects the code in general labels Nov 12, 2024
@llucax llucax enabled auto-merge November 12, 2024 09:23
@llucax llucax added this to the v1.0.0 milestone Nov 12, 2024
@llucax llucax self-assigned this Nov 12, 2024
@llucax llucax added type:bug Something isn't working type:enhancement New feature or enhancement visitble to users labels Nov 12, 2024
@llucax llucax added this pull request to the merge queue Nov 12, 2024
Merged via the queue into frequenz-floss:v1.x.x with commit b2aa061 Nov 12, 2024
16 checks passed
@llucax llucax deleted the marsh-reactive branch November 12, 2024 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:code Affects the code in general part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests type:bug Something isn't working type:enhancement New feature or enhancement visitble to users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants