Skip to content

Conversation

@knmcguire
Copy link
Contributor

In PR #540 I've placed these lines in the wrong order. Ofcourse it needs to be done after the commit (thanks @esteve). The dangers of doing blind PRs testing the action itself :)

Let's try again!

@knmcguire
Copy link
Contributor Author

knmcguire commented Oct 26, 2025

running it on my fork currently: https://github.com/knmcguire/ros2_rust/actions/workflows/generate-bindings.yml

It will fail the push for sure (doesn't have the rights), but at least I can see if the rebase works

@knmcguire knmcguire marked this pull request as ready for review October 26, 2025 09:16
@knmcguire
Copy link
Contributor Author

success! All good to go!

Copy link
Collaborator

@esteve esteve left a comment

Choose a reason for hiding this comment

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

@knmcguire thanks!

@esteve esteve merged commit b4a5e32 into ros2-rust:main Oct 26, 2025
9 checks passed
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.

3 participants