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

output boolean type for standalone and aggregate #46

Merged
merged 4 commits into from
Mar 16, 2013
Merged

output boolean type for standalone and aggregate #46

merged 4 commits into from
Mar 16, 2013

Conversation

jlambert121
Copy link
Contributor

No description provided.

@jamtur01
Copy link
Contributor

I think there's something a big ugly about it and I'd love @rodjek's view on it but I can't think of a better way so. :(

@jlambert121
Copy link
Contributor Author

agree it is really ugly but puppet appears to be using the symbols :true and :false. I tried updating the type so it was using the params true and false rather than the symbols without any change. I'll push a fix for the provider updating every run and we can see what @rodjek thinks as well.

jamtur01 added a commit that referenced this pull request Mar 16, 2013
output boolean type for standalone and aggregate
@jamtur01 jamtur01 merged commit bc00bab into sensu:master Mar 16, 2013
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