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

Added help command for duckpan #168

Merged
merged 1 commit into from
Nov 11, 2014
Merged

Conversation

killerfish
Copy link
Contributor

As commented in #166.

@moollaza
Copy link
Member

Just commented in #166, but duckpan help already works for me as expected, though it isn't implemented as a specific Cmd. I believe that was by design so we could easily use it to show the help output for unknown commands.

This should be okay though because we're just telling them to use duckpan help now for unknown commands.

@killerfish
Copy link
Contributor Author

@moollaza can you try with the latest build, because help wasnt working for me, apparently (no pod just the error msg), after i merged with the latest branch. The reason i mentioned in #166 made sense to me, so a bit curious, how help could trigger.

@moollaza
Copy link
Member

@killerfish apologies, you're correct. I haven't released the latest changes yet but in Master because that line was removed, duckpan help no longer works.

@moollaza
Copy link
Member

@killerfish this LGTM. I'm going to make another PR to build on this. Adds tests and fixed duckpan to also output the help info :)

moollaza added a commit that referenced this pull request Nov 11, 2014
Added help command for duckpan
@moollaza moollaza merged commit 4ceb4eb into duckduckgo:master Nov 11, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants