Skip to content

[docs] Update reporting docs#739

Merged
mx-psi merged 2 commits intomainfrom
mx-psi/update-dogapi-docs
Jun 3, 2022
Merged

[docs] Update reporting docs#739
mx-psi merged 2 commits intomainfrom
mx-psi/update-dogapi-docs

Conversation

@mx-psi
Copy link
Copy Markdown
Member

@mx-psi mx-psi commented Jun 3, 2022

What does this PR do?

Update reporting docs to remove step for installing the dogapi gem.

Motivation

The docs told users to install the dogapi gem manually, but this is managed by the module since #613. Following the docs results in a duplicate resource declaration.

@mx-psi mx-psi changed the title [docs] Update dogapi docs [docs] Update reporting docs Jun 3, 2022
@mx-psi mx-psi marked this pull request as ready for review June 3, 2022 10:23
@mx-psi mx-psi requested review from a team as code owners June 3, 2022 10:23
Copy link
Copy Markdown
Contributor

@kayayarai kayayarai left a comment

Choose a reason for hiding this comment

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

One minor edit for verb tense, but 👍

Comment thread README.md Outdated
Co-authored-by: Kari Halsted <12926135+kayayarai@users.noreply.github.com>
@mx-psi mx-psi merged commit a1cde85 into main Jun 3, 2022
@mx-psi mx-psi deleted the mx-psi/update-dogapi-docs branch June 3, 2022 15:43
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Feb 5, 2026
* [docs] Update reporting docs

* Apply suggestions from code review

Co-authored-by: Kari Halsted <12926135+kayayarai@users.noreply.github.com>

Co-authored-by: Kari Halsted <12926135+kayayarai@users.noreply.github.com>
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