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 ruby agent configuration #2694

Merged
merged 4 commits into from
Jun 18, 2021

Conversation

tannalynn
Copy link
Contributor

@tannalynn tannalynn commented Jun 14, 2021

Give us some context

This PR updates our autogenerated config documentation. This updates the wording on browser_monitoring.auto_instrument config option to clarify the default value, also we updated links to use markdown instead of html.
I also added html comments to mark what sections are auto generated from the agent so it's easier to tell. (There used to be comments that showed the section was generated, but it seems like it went missing since the move to git for the docs site)

Are you making a change to site code?

no

@github-actions github-actions bot added this to Needs Triage in Docs PRs and Issues Jun 14, 2021
@dbarnesbrownNR dbarnesbrownNR self-assigned this Jun 14, 2021
@dbarnesbrownNR dbarnesbrownNR added content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers) labels Jun 14, 2021
@dbarnesbrownNR dbarnesbrownNR moved this from Needs Triage to In review (Hero or any TW) in Docs PRs and Issues Jun 15, 2021
Copy link
Contributor

@dbarnesbrownNR dbarnesbrownNR left a comment

Choose a reason for hiding this comment

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

All good, merging!

@dbarnesbrownNR dbarnesbrownNR merged commit ca470bd into newrelic:develop Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers)
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants