Skip to content

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Jan 29, 2014

No description provided.

@bdunne
Copy link
Member Author

bdunne commented Jan 29, 2014

@Fryguy Please review 🍰

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 731422c on brandondunne:enable_disable_and_list_rhn_and_subscription_manager_repos into a93bcab on ManageIQ:master.

spec/rhn_spec.rb Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unindent these parens.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In fact all of the tailing parens below have not been unindented properly.

@bdunne
Copy link
Member Author

bdunne commented Jan 30, 2014

@Fryguy Updated.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling b59f3df on brandondunne:enable_disable_and_list_rhn_and_subscription_manager_repos into a93bcab on ManageIQ:master.

@bdunne
Copy link
Member Author

bdunne commented Jan 31, 2014

Don't merge yet... found an issue with Rhn compatability

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 61d4739 on brandondunne:enable_disable_and_list_rhn_and_subscription_manager_repos into a93bcab on ManageIQ:master.

@bdunne
Copy link
Member Author

bdunne commented Jan 31, 2014

Incompatibility fixed

Fryguy added a commit that referenced this pull request Jan 31, 2014
…and_subscription_manager_repos

Enable, Disable, and List Rhn and SubscriptionManager repos
@Fryguy Fryguy merged commit 0e76241 into ManageIQ:master Jan 31, 2014
@bdunne bdunne deleted the enable_disable_and_list_rhn_and_subscription_manager_repos branch February 1, 2014 02:44
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.

3 participants