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

Improve tracing configuration error reporting #85

Merged

Conversation

christophebedard
Copy link
Member

@christophebedard christophebedard commented Jan 10, 2024

Provide a bit more information in error messages.

Also, add a test for invalid context domains with the Trace action.

Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
@christophebedard christophebedard force-pushed the christophebedard/improve-ros2-trace-error-reporting branch from 8e203f6 to 3b6aecd Compare February 12, 2024 16:11
@christophebedard
Copy link
Member Author

@mjcarroll friendly ping

@christophebedard
Copy link
Member Author

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@christophebedard christophebedard merged commit 734d867 into rolling Feb 12, 2024
9 checks passed
@christophebedard christophebedard deleted the christophebedard/improve-ros2-trace-error-reporting branch February 12, 2024 21:54
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.

None yet

2 participants