-
Notifications
You must be signed in to change notification settings - Fork 889
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
Add yaml semantic conventions for Exception #927
Add yaml semantic conventions for Exception #927
Conversation
@thisthat Please rebase and resolve the conflicts ;) |
0ac297c
to
001e9ce
Compare
@carlosalberto done in 22bbb9b :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocking just to understand clearly the intent regarding exception.stacktrace
. The PR itself is highly welcome, but I think the way the example is rendered makes it confusing.
Indeed it seems like the \n are not doing what they should inside the ''. |
@tigrannajaryan could you please explain better the problem? Do you want the example to be encapsulated in quotes? |
I think the original example was also misleading. I expect not to see any literal \n characters in the example and I expect not to see backslaches before double quote characters. |
@tigrannajaryan I've created #951 to track the example change |
Changes
Updated Exception semantic convention to the YAML format.
Related issues #547