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

Put back region_description method that was accidentally extracted #15372

Merged
merged 1 commit into from
Jun 14, 2017

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented Jun 14, 2017

Depends on ManageIQ/activerecord-id_regions#3 and subsequent gem release EDIT: Now merged and released as 0.2.0

I accidentally moved this method to ActiveRecord::IdRegions, but it could never work there standalone because it depends on the miq_region method defined in ManageIQ. This PR puts it back.

@jrafanie Please review.

@miq-bot
Copy link
Member

miq-bot commented Jun 14, 2017

Checked commit Fryguy@2fac75c with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
3 files checked, 0 offenses detected
Everything looks fine. 🏆

@Fryguy Fryguy closed this Jun 14, 2017
@Fryguy Fryguy reopened this Jun 14, 2017
@Fryguy Fryguy changed the title [Depends on ManageIQ/activerecord-id_regions#3] Put back region_description method that was accidentally extracted Put back region_description method that was accidentally extracted Jun 14, 2017
@jrafanie jrafanie merged commit f17e815 into ManageIQ:master Jun 14, 2017
@jrafanie jrafanie added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 14, 2017
@Fryguy Fryguy deleted the put_back_extracted_method branch June 14, 2017 20:48
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.

3 participants