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

Fix timeout decorator #6

Merged
merged 5 commits into from
Aug 8, 2024
Merged

Fix timeout decorator #6

merged 5 commits into from
Aug 8, 2024

Conversation

nandojve
Copy link
Contributor

@nandojve nandojve commented Aug 6, 2024

No description provided.

Signed-off-by: Gerson Fernando Budke <[email protected]>
@nandojve nandojve requested a review from otavio August 6, 2024 20:01
@otavio
Copy link
Member

otavio commented Aug 6, 2024

@@ -10,6 +10,7 @@ zephyr_sources_ifdef(CONFIG_ZEPHYR_BEHAVIOUR_TREE
)

zephyr_sources_ifdef(CONFIG_ZEPHYR_BEHAVIOUR_TREE_NODE_CONTEXT
zephyrbt_decorator_delay.c
Copy link
Member

Choose a reason for hiding this comment

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

I believe the correct name is Sleep. You can change the name of the decorator or change the code generation to adjust the code of the decorator when generating it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@nandojve nandojve added the DNM Do Not Merge label Aug 8, 2024
Signed-off-by: Gerson Fernando Budke <[email protected]>
Signed-off-by: Gerson Fernando Budke <[email protected]>
Signed-off-by: Gerson Fernando Budke <[email protected]>
Signed-off-by: Gerson Fernando Budke <[email protected]>
@nandojve nandojve requested a review from otavio August 8, 2024 17:44
@nandojve nandojve removed the DNM Do Not Merge label Aug 8, 2024
@otavio otavio merged commit 3643c9e into master Aug 8, 2024
2 checks passed
@otavio otavio deleted the fix_timeout_decorator branch August 8, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants