Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Split the rho clicommands.py file into individual class files and uti… #58

Merged
merged 1 commit into from
Jul 12, 2017
Merged

Split the rho clicommands.py file into individual class files and uti… #58

merged 1 commit into from
Jul 12, 2017

Conversation

chambridge
Copy link

…lity files.

@coveralls
Copy link

coveralls commented Jul 11, 2017 •
edited

Coverage Status

Coverage increased (+2.4%) to 51.322% when pulling cf5130c on issues/38-part-4 into 53cbffa on master.

@chambridge chambridge mentioned this pull request Jul 11, 2017
Closed
elyezer
elyezer approved these changes Jul 11, 2017
View reviewed changes
Copy link

@elyezer elyezer left a comment

Choose a reason for hiding this comment

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

It seems to be just a code refactoring to relocate some of the code. Looks good to me.

@chambridge chambridge merged commit f95cbb8 into master Jul 12, 2017
@chambridge chambridge deleted the issues/38-part-4 branch July 12, 2017 13:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@elyezer elyezer elyezer approved these changes

Assignees

@elyezer elyezer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@chambridge @coveralls @elyezer