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.

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

Add command for dbconsole #126

Closed
wants to merge 1 commit into from

Conversation

unrealhoang
Copy link
Contributor

Status: In Progress
Taker: @Linuus

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.33%) when pulling da3398a on unrealhoang:command_for_dbconsole into 9e7ec45 on lotus:master.

method_option :help, aliases: '-h', desc: 'displays the usage method'
def dbconsole(app)
if options[:help]
invoke :help, ['new']
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should invoke help for dbconsole here.

@jodosha jodosha added the waiting label Jan 9, 2015
@runlevel5 runlevel5 mentioned this pull request Jan 17, 2015
@Linuus
Copy link
Contributor

Linuus commented Jan 17, 2015

@unrealhoang Are you still working on this or is it ok that I take a stab at it?

We were discussing this here #123

Cheers!

@unrealhoang
Copy link
Contributor Author

@Linuus It would be great if you take on it. I still not have free time to battle with it yet :(

@Linuus
Copy link
Contributor

Linuus commented Jan 18, 2015

Ok, I'll have a look at it :)

Ping @joneslee85

On Sun, Jan 18, 2015 at 1:56 PM, unrealhoang notifications@github.com
wrote:

@Linuus It would be great if you take on it. I still not have free time to battle with it yet :(

Reply to this email directly or view it on GitHub:
#126 (comment)

@runlevel5
Copy link
Member

@Linuus @unrealhoang okay, I reassign this to @Linuus

@jodosha
Copy link
Member

jodosha commented Feb 13, 2015

Closing in favor of #152

@jodosha jodosha closed this Feb 13, 2015
@jodosha jodosha removed the waiting label Feb 13, 2015
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.

None yet

6 participants