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_not_found_handler for fish shell #20

Merged
merged 1 commit into from Sep 14, 2014

Conversation

CIAvash
Copy link
Contributor

@CIAvash CIAvash commented Sep 14, 2014

I tried to keep it similar to the other scripts. But I didn't return anything because fish outputs fish: Unknown command 'command-name' and returns 127 in the end.

@falconindy
Copy link
Owner

This won't install the hook the doc dir, making it not so useful. Could you amend your change to wire this up to the makefile? It should be a one liner...

@CIAvash
Copy link
Contributor Author

CIAvash commented Sep 14, 2014

Changed Makefile and README. Is this correct?

@falconindy
Copy link
Owner

Looks correct at a glance, but the computer can tell you a lot more. Does the built manpage look correct (POD errors show up at the bottom of the compiled page)? Does 'make distcheck' pass?

@CIAvash
Copy link
Contributor Author

CIAvash commented Sep 14, 2014

Yes, everything looks good.

falconindy added a commit that referenced this pull request Sep 14, 2014
Add command_not_found_handler for fish shell
@falconindy falconindy merged commit 387bba4 into falconindy:master Sep 14, 2014
@falconindy
Copy link
Owner

Thanks!

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

2 participants