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.

Sign up for GitHub

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

Jump to bottom

[WIP] Refactor 0607 #3

Merged
merged 16 commits into from
Jun 9, 2014
Merged

[WIP] Refactor 0607 #3

merged 16 commits into from
Jun 9, 2014

Conversation

masa21kik
Copy link
Owner

  • fix rubocop detections
  • fix reek detections
  • refactor to command class

@masa21kik masa21kik changed the title Refact 0607 [WIP] Refact 0607 Jun 7, 2014
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling f52fb62 on refact-0607 into 0873698 on master.

@masa21kik masa21kik changed the title [WIP] Refact 0607 [WIP] Refactor 0607 Jun 8, 2014
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 32a2336 on refact-0607 into 0873698 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 49a6f8e on refact-0607 into 0873698 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 7f47db4 on refact-0607 into 0873698 on master.

masa21kik added a commit that referenced this pull request Jun 9, 2014
@masa21kik
Merge pull request #3 from masa21kik/refact-0607
4b1a13e 
[WIP] Refactor 0607
@masa21kik masa21kik merged commit 4b1a13e into master Jun 9, 2014
@masa21kik masa21kik deleted the refact-0607 branch June 9, 2014 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@masa21kik @coveralls