ros2
Add warning that actions require ament_cmake, not ament_python #4986
Looks like @Yadunund and @fujitatomoya's suggestions are sufficient. I do want to point out that it helps us a lot if you can include a one or two sentence description of your pull request.
I just approved #5050 which should make #4985 a lot easier. Once you wrap up this PR you are more than welcome to give that one a try once you read my comment there.
@vimal0athithan friendly ping. Can you accepted the suggestions and make the necessary changes please so we can get this out the door?
@kscottz @Yadunund @fujitatomoya I have applied the suggested changes. Please review the updated version.
@kscottz Thank you for the ping! I apologize for always contributing late always —I'm a university student, and my exams are currently ongoing, which has affected my availability. I’ve accepted the suggestions and made the necessary changes as requested. Please review the updates. Sorry again for the delay!
@kscottz @Yadunund @fujitatomoya, I’ve pushed updates to fix the linting and test failures based on the check logs. Please review my new PR changes. Thank you!
@kscottz @Yadunund @fujitatomoya, I’ve fixed the trailing whitespace and missing newline issues. Please review again. Thanks!
@vimal0athithan friendly ping. We just have a couple of change requests to work through.