Skip to content

Conversation

@maxinspace
Copy link

Finished. Everything works and fine.

description commit #2 Added coloring text as --c color. Will refactor it into another class after finishing --verbose.
gem colorize is working fine.

description commit #1 Made yet not working class commander, where all logic for commands is going to be, added gem colorize for further coloring, and now terminal shows amount of users and repos.

@maxinspace maxinspace changed the title Started work, created commander class && 1-easy task done Max Larionov - Github-team-remover progress Apr 23, 2014
Copy link
Contributor

Choose a reason for hiding this comment

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

@des1tro could you please create separate class for reporting this information?

Remover::Report.new(unused_team).report

Copy link
Contributor

Choose a reason for hiding this comment

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

Please create separata class for outputting invormation.
Logic should be incapsulated there.

Copy link
Author

Choose a reason for hiding this comment

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

ok, will do it today and global refactoring.

Copy link
Author

Choose a reason for hiding this comment

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

Got really huge troubles with making another class, code not working on my PC, but works on another.

@maxinspace maxinspace changed the title Max Larionov - Github-team-remover progress Max Larionov - Github-team-remover progress(almost done) Apr 26, 2014
@maxinspace maxinspace changed the title Max Larionov - Github-team-remover progress(almost done) Max Larionov - Github-team-remover progress(Finished.) Apr 29, 2014
Updated readme
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