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

Replace PuppetX::Voxpupuli::Corosync::Provider:: by self.class #431

Merged

Conversation

btravouillon
Copy link
Member

Fix #376

@bastelfreak
Copy link
Member

Can you take a look at the faiing spec tests?

@btravouillon
Copy link
Member Author

WIP: please don't merge

@alexjfisher alexjfisher changed the title Replace PuppetX::Voxpupuli::Corosync::Provider:: by self.class WIP: Replace PuppetX::Voxpupuli::Corosync::Provider:: by self.class Jan 15, 2018
@bastelfreak
Copy link
Member

Hi @actatux, what is the current state here?

@bastelfreak bastelfreak added the needs-feedback Further information is requested label Mar 28, 2018
@btravouillon
Copy link
Member Author

You can merge if the review is good.

@bastelfreak bastelfreak force-pushed the replace_puppetx_by_self_class branch from e750ca2 to 7149142 Compare March 28, 2018 23:06
@bastelfreak
Copy link
Member

I rebased the branch again. Will merge it when it turn green.

@bastelfreak bastelfreak merged commit 132b3c1 into voxpupuli:master Mar 29, 2018
@bastelfreak bastelfreak changed the title WIP: Replace PuppetX::Voxpupuli::Corosync::Provider:: by self.class Replace PuppetX::Voxpupuli::Corosync::Provider:: by self.class Mar 29, 2018
@bastelfreak bastelfreak removed needs-feedback Further information is requested needs-squash labels Mar 29, 2018
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.

2 participants