Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testing with storm #6

Merged
merged 7 commits into from
Jun 26, 2024
Merged

Testing with storm #6

merged 7 commits into from
Jun 26, 2024

Conversation

ct2034
Copy link
Member

@ct2034 ct2034 commented Jun 25, 2024

No description provided.

@ct2034 ct2034 marked this pull request as ready for review June 25, 2024 12:26
Signed-off-by: Christian Henkel <[email protected]>
@ct2034 ct2034 requested a review from MarcoLm993 June 25, 2024 12:28
Signed-off-by: Christian Henkel <[email protected]>
Copy link
Collaborator

@MarcoLm993 MarcoLm993 left a comment

Choose a reason for hiding this comment

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

Just a small comment, but everything looks molto bello! :)

@@ -0,0 +1,30 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Jani supports also listing multiple properties in the array, as long as they have different names. In smc_storm you can (and for now have to) select one to verify, based on the assigned name.

Maybe it would be easier to have only a battery_depleted_properties.jani listing all available properties, and then calling smc_storm multiple times, each time with a different property.

This way, you do not have to add another main xml file

Signed-off-by: Christian Henkel <[email protected]>
@ct2034 ct2034 merged commit 79020ec into main Jun 26, 2024
2 checks passed
@ct2034 ct2034 deleted the add/testing_with_storm branch June 26, 2024 15:03
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