Skip to content

Conversation

@ruthnaebeck
Copy link
Contributor

@ruthnaebeck ruthnaebeck commented Mar 13, 2020

What does this PR do?

  • Audit README
  • Move development content into CONTRIBUTING.md
  • Add CODEOWNERS

Motivation

OKR - To have the README pull into the public docs site.

Preview link

N/A

@ruthnaebeck ruthnaebeck requested review from a team March 27, 2020 17:23
@ruthnaebeck ruthnaebeck changed the title [WIP] DOCS-630 Ansible docs DOCS-630 Ansible docs Mar 27, 2020
Copy link
Contributor

@KSerrania KSerrania left a comment

Choose a reason for hiding this comment

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

LGTM overall, made some suggestions to change outdated / wrong details

Co-Authored-By: Kylian Serrania <kylian.serrania@datadoghq.com>
Copy link
Contributor

@KSerrania KSerrania left a comment

Choose a reason for hiding this comment

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

🚀

@ruthnaebeck ruthnaebeck requested review from cswatt and removed request for cswatt April 7, 2020 16:02
@KSerrania
Copy link
Contributor

Hey @ruthnaebeck, is this ready to merge? I'd like to release a new version of the role today or tomorrow.

Co-Authored-By: cswatt <cecilia.watt@datadoghq.com>
@ruthnaebeck
Copy link
Contributor Author

@KSerrania - ready to go. Just a few wording fixes.

Co-Authored-By: cswatt <cecilia.watt@datadoghq.com>
@ruthnaebeck ruthnaebeck merged commit 80aae10 into master Apr 7, 2020
@ruthnaebeck ruthnaebeck deleted the ruth/docs-630 branch April 7, 2020 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants