-
Notifications
You must be signed in to change notification settings - Fork 598
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
Update README for clarity and consistency #1034
Conversation
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.
Great stuff! Brought up a few suggestions.
@@ -8,7 +8,7 @@ You can also extend the agent's performance monitoring to | |||
[collect and analyze business data](https://docs.newrelic.com/docs/agents/ruby-agent/getting-started/introduction-new-relic-ruby#business-data) | |||
to help you improve the customer experience and make data-driven business decisions. | |||
|
|||
The New Relic Ruby Agent is dual-purposed as a either a Gem or a Rails plugin, | |||
The New Relic Ruby agent is dual-purposed as either a Gem or a Rails plugin, |
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.
A
-> a
seems to put us in alignment with the majority of the other agents. Looks like Python's still rocking an A
. The change makes sense. 👍
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.
Looks great! Thanks for getting the readme updated 🎉
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.
👍
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.
Thanks for taking this on, Hannah! I have a few recommendations, but I'm also happy to approve as-is. 🌱
67ba32c
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.
Looks great! 👍
Before contributing, please read our contributing guidelines and code of conduct.
Overview
Updated capitalization of "New Relic Ruby agent" to be consistent with marketing materials. Added punctuation and defined "CLA" for clarity. Updated README sign-off.
Related Github Issue
Include a link to the related GitHub issue, if applicable
Testing
The agent includes a suite of unit and functional tests which should be used to
verify your changes don't break existing functionality. These tests will run with
Github Actions when a pull request is made. More details on running the tests locally can be found
here for our unit tests,
and here for our functional tests.
For most contributions it is strongly recommended to add additional tests which
exercise your changes.
Reviewer Checklist