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

Factory names are usually symbols #16975

Merged
merged 1 commit into from
Feb 8, 2018

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Feb 8, 2018

No description provided.

@chrisarcand
Copy link
Member

@miq-bot merge when green

...hey man I'm going to keep trying this until someone implements it.

@miq-bot
Copy link
Member

miq-bot commented Feb 8, 2018

@chrisarcand unrecognized command 'merge', ignoring...

Accepted commands are: add_label, assign, close_issue, move_issue, remove_label, rm_label, set_milestone

@miq-bot
Copy link
Member

miq-bot commented Feb 8, 2018

Checked commit bdunne@9b70377 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@Fryguy
Copy link
Member

Fryguy commented Feb 8, 2018

I think I'll add a merge command to the bot that just responds with "I'm sorry @chrisarcand, I'm afraid I can't do that" https://www.youtube.com/watch?v=9W5Am-a_xWw

@chrisarcand chrisarcand merged commit 863255b into ManageIQ:master Feb 8, 2018
@chrisarcand chrisarcand added this to the Sprint 79 Ending Feb 12, 2018 milestone Feb 8, 2018
@chrisarcand chrisarcand self-assigned this Feb 8, 2018
@bdunne bdunne deleted the factory_references branch February 8, 2018 22:04
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.

4 participants