-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[cart] document instrumentation #390
Conversation
@puckpuck what are your thoughts on adding language to the file names / titles? I feel like a user would prefer to see a language and go directly there rather than consult our architecture for direction |
Seems like a great idea! The language could also be mentioned in the top section of each doc, for example, next to the source code link. |
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
I think this is a great idea.
I will also add it to the title for each one also. I need to make a few small changes to this PR so will do that here, and submit another one to clean up all the other docs we wrote thus far. |
Awesome! |
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. Please do replace auto-instrumentation with instrumentation library.
We'll want to update this in all service docs but can wait until we finish adding them |
* cart service doc details * clean up docs Co-authored-by: Carter Socha <43380952+cartersocha@users.noreply.github.com>
Adds documentation for cart service, focused on instrumentation code.