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

Stop dropping logs all over the place... #1294

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Conversation

@ruffsl ruffsl requested a review from a team as a code owner July 19, 2024 18:29
Copy link
Member

@ooeygui ooeygui left a comment

Choose a reason for hiding this comment

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

Good catch - thank you for the contribution!

@ruffsl
Copy link
Author

ruffsl commented Jul 22, 2024

@ooeygui looks like all CI is passing. Could we cut a minor/patch release?

@ooeygui ooeygui merged commit 35f2f3b into ms-iot:master Jul 22, 2024
6 checks passed
@ooeygui
Copy link
Member

ooeygui commented Jul 22, 2024

Thank you for the contribution. I could use some help testing the release. I have one big issue I'm working on related to URDF before we can push it.

@ruffsl
Copy link
Author

ruffsl commented Jul 22, 2024

I could use some help testing the release. I have one big issue I'm working on related to URDF before we can push it.

Sure, got a link to the ticket? Didn't see any recent URDF related PRs via search.

@ooeygui
Copy link
Member

ooeygui commented Jul 22, 2024

@ruffsl Thank you! I'm working on it in the babylon_ros repo, which is what this extension depends on. Here's the bug I'm iterating on #1139

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