Skip to content

revise help text #7

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

Merged
merged 1 commit into from
Oct 26, 2020
Merged

revise help text #7

merged 1 commit into from
Oct 26, 2020

Conversation

sdarwin
Copy link
Collaborator

@sdarwin sdarwin commented Oct 26, 2020

  • Update help text
  • Add support for an "everything" keyword search

@sdarwin sdarwin requested a review from JankoDedic October 26, 2020 16:23
@JankoDedic
Copy link
Collaborator

I agree with the help and "search everything" changes. However, I don't see the point of handling the "blank" command this way. I don't think we need to print anything for failure since failure is obvious and an unwanted help message just pollutes the chat.

@JankoDedic JankoDedic merged commit 0d61264 into master Oct 26, 2020
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.

2 participants