Skip to content

added perform in WaitForIMU Startup and WaitForPressureStartup#718

Merged
ayin21 merged 2 commits intomainfrom
fix/perfom-in-AbstractHCMActionElements-override
Sep 4, 2025
Merged

added perform in WaitForIMU Startup and WaitForPressureStartup#718
ayin21 merged 2 commits intomainfrom
fix/perfom-in-AbstractHCMActionElements-override

Conversation

@ayin21
Copy link
Copy Markdown
Contributor

@ayin21 ayin21 commented Sep 2, 2025

Summary

There was a not implemented error as an abstract super class wants an implemented perform method, which was not implemented.

Proposed changes

Related issues

Checklist

  • Run colcon build
  • Write documentation
  • Test on your machine
  • Test on the robot
  • Create issues for future work
  • Triage this PR and label it

@github-project-automation github-project-automation Bot moved this to 🆕 New in Software Sep 2, 2025
@ayin21 ayin21 requested a review from Flova September 2, 2025 17:54
@github-project-automation github-project-automation Bot moved this from 🆕 New to 👀 In review in Software Sep 3, 2025
@Flova
Copy link
Copy Markdown
Member

Flova commented Sep 3, 2025

Please run make format before merge (see failed check)

@ayin21 ayin21 merged commit e687d7a into main Sep 4, 2025
1 of 2 checks passed
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Software Sep 4, 2025
@ayin21 ayin21 deleted the fix/perfom-in-AbstractHCMActionElements-override branch September 4, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants