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.

Sign up for GitHub

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

Jump to bottom

Update RabbitMQ README #9326

Merged
merged 1 commit into from
May 10, 2021
Merged

Update RabbitMQ README #9326

merged 1 commit into from
May 10, 2021

Conversation

ruthnaebeck
Copy link
Contributor

What does this PR do?

  • Remove partial YAML from RabbitMQ README.
  • Wording update to logs section.

Motivation

Customer assumed only rabbitmq_api_url was required.

Additional Notes

N/A

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached

@ruthnaebeck ruthnaebeck self-assigned this May 10, 2021
@ruthnaebeck ruthnaebeck requested review from a team as code owners May 10, 2021 19:36
@kayayarai kayayarai merged commit 87a913b into master May 10, 2021
@kayayarai kayayarai deleted the ruth/docs-1808 branch May 10, 2021 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kayayarai kayayarai kayayarai approved these changes

Assignees

@ruthnaebeck ruthnaebeck

Labels
changelog/no-changelog documentation integration/rabbitmq
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ruthnaebeck @kayayarai