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

Command line console tool #121

Merged
merged 1 commit into from Jan 16, 2015
Merged

Command line console tool #121

merged 1 commit into from Jan 16, 2015

Conversation

hyades
Copy link
Member

@hyades hyades commented Jan 12, 2015

No description provided.

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.64% when pulling 6c37688 on hyades:merge_console_1 into 34a2532 on timvideos:master.

@mithro mithro mentioned this pull request Jan 14, 2015
Closed
@mithro mithro changed the title Merge console 1 Command line console tool Jan 14, 2015
@mithro mithro added this to the At TimVideos Sprints & LCA milestone Jan 14, 2015
mithro
mithro reviewed Jan 16, 2015
View reviewed changes
console.py
try:
val = self.SWITCH_MAPPING[line.lower()]
except KeyError:
print "Invalid"
Copy link
Member

Choose a reason for hiding this comment

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

Need a return here.

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@hyades hyades force-pushed the merge_console_1 branch 3 times, most recently from 8d86bc6 to 0c72371 Compare January 16, 2015 07:15
@landscape-bot
Copy link

Code Health
Repository health decreased by 1% when pulling 0c72371 on hyades:merge_console_1 into 53415bc on timvideos:master.

mithro added a commit that referenced this pull request Jan 16, 2015
@mithro
Merge pull request #121 from hyades/merge_console_1
80951da 
Command line console tool
@mithro mithro merged commit 80951da into timvideos:master Jan 16, 2015
@mithro mithro deleted the merge_console_1 branch January 16, 2015 07:30
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
enhancement python-api server
Projects
None yet
Milestone
At TimVideos Sprints & LCA
3 participants
@hyades @landscape-bot @mithro