Skip to content
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

Merged
merged 4 commits into from
Mar 25, 2022
Merged

Update README for clarity and consistency #1034

merged 4 commits into from
Mar 25, 2022

Conversation

hannahramadan
Copy link
Contributor

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

  • Perform code review
  • Add performance label
  • Perform appropriate level of performance testing
  • Confirm all checks passed
  • Add version label prior to acceptance

Copy link
Contributor

@fallwith fallwith left a 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,
Copy link
Contributor

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. 👍

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
tannalynn
tannalynn previously approved these changes Mar 23, 2022
Copy link
Contributor

@tannalynn tannalynn left a 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 🎉

fallwith
fallwith previously approved these changes Mar 24, 2022
Copy link
Contributor

@fallwith fallwith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

kaylareopelle
kaylareopelle previously approved these changes Mar 24, 2022
Copy link
Contributor

@kaylareopelle kaylareopelle left a 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. 🌱

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
fallwith
fallwith previously approved these changes Mar 24, 2022
Copy link
Contributor

@fallwith fallwith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! 👍

@fallwith fallwith merged commit 3b4436f into newrelic:dev Mar 25, 2022
@fallwith fallwith mentioned this pull request Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants