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

Updates to rabbitmq::server docs #745

Merged
merged 1 commit into from
Dec 7, 2018

Conversation

wyardley
Copy link
Contributor

@wyardley wyardley commented Dec 7, 2018

Pull Request (PR) description

Per discussion in #744, document rabbitmq::server, just because it's so messy.

That said, I'm thinking we might want to deprecate / remove this compatibility interface?

@wyardley wyardley added the docs Improvements or additions to documentation label Dec 7, 2018
@wyardley wyardley requested a review from ekohl December 7, 2018 03:17
@wyardley wyardley force-pushed the wyardley-docs-update-2-service branch 3 times, most recently from b0547de to 4e03fe7 Compare December 7, 2018 03:28
@wyardley wyardley force-pushed the wyardley-docs-update-2-service branch from 4e03fe7 to 0adc07f Compare December 7, 2018 03:38
@ekohl
Copy link
Member

ekohl commented Dec 7, 2018

We could also nuke the class. It's been a compat wrapper since 2014.

@wyardley
Copy link
Contributor Author

wyardley commented Dec 7, 2018

@ekohl Yeah, my thoughts too. But since I fixed the docs, let’s go ahead and release w/ this, and kill it on the next major (if we remember)?

@wyardley wyardley merged commit 2cc0193 into voxpupuli:master Dec 7, 2018
@wyardley wyardley deleted the wyardley-docs-update-2-service branch December 7, 2018 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants