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

Add support for Hash definition #9

Merged
merged 1 commit into from
May 22, 2015

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented May 22, 2015

Closes #7

@jerryk55 @chessbyte Please review.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 99.53% when pulling 4640bea on Fryguy:add_hash_support_for_definition into 80e6a26 on ManageIQ:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 99.53% when pulling 4640bea on Fryguy:add_hash_support_for_definition into 80e6a26 on ManageIQ:master.

@Fryguy Fryguy force-pushed the add_hash_support_for_definition branch from 4640bea to f47b18a Compare May 22, 2015 18:25
@Fryguy
Copy link
Member Author

Fryguy commented May 22, 2015

Rebase fail...It should be fixed now.

@miq-bot
Copy link
Member

miq-bot commented May 22, 2015

Checked commit Fryguy@f47b18a with rubocop 0.27.1
2 files checked, 0 offenses detected
Everything looks good. ⭐

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 99.53% when pulling f47b18a on Fryguy:add_hash_support_for_definition into 80e6a26 on ManageIQ:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 99.53% when pulling f47b18a on Fryguy:add_hash_support_for_definition into 80e6a26 on ManageIQ:master.

@jerryk55
Copy link
Member

Looks good. Simple enough.

Fryguy added a commit that referenced this pull request May 22, 2015
@Fryguy Fryguy merged commit 242b530 into ManageIQ:master May 22, 2015
@Fryguy Fryguy deleted the add_hash_support_for_definition branch May 22, 2015 19:40
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.

Support Hash syntax for the definition.
4 participants