-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 native library instrumentation guidance #783
Add native library instrumentation guidance #783
Conversation
Looking for early feedback on the native library instrumentation doc. /cc @tedsuo |
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.
lgtm in general, couple suggestions. There are sitll quite a few TODO, but looks good as an initial version with the placeholder. If it will be merged, I'd suggest to replace "TODO" with "Under Constructions" node and emoji 🚧, so it will be reader-friendly - indicating there is work ongoing rather than the article is incomplete.
Co-authored-by: Sergey Kanzhelev <S.Kanzhelev@live.com>
Thanks @SergeyKanzhelev |
0c8dafb
to
c8ec685
Compare
c8ec685
to
742ee79
Compare
thanks @austinlparker ! |
No description provided.