-
Notifications
You must be signed in to change notification settings - Fork 225
docs: Enhance README #1711
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
docs: Enhance README #1711
Conversation
kaylareopelle
left a comment
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.
Thank you for this PR! The text looks good. I have some feedback on the placement.
Lines 44-50 should be above the configuration section. They go along with the earlier code snippet related to installing a single instrumentation.
Also, what do you think about adding a Configuration/Configuration Options heading, or something to that effect? It would keep the document's structure more consistent.
cb5f73c to
1e15190
Compare
Document configuration options for ActiveJob instrumentation.
355def8 to
2d38afc
Compare
kaylareopelle
left a comment
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.
Thank you!
Enhance README with additional configuration options for ActiveJob instrumentation.