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

test precmd_function without test compound command and command substitution #15

Merged
merged 1 commit into from
Nov 1, 2015
Merged

test precmd_function without test compound command and command substitution #15

merged 1 commit into from
Nov 1, 2015

Conversation

d630
Copy link
Contributor

@d630 d630 commented Oct 24, 2015

Tested with:

% bash --version
GNU bash, version 4.3.42(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2013 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

@d630
Copy link
Contributor Author

d630 commented Oct 27, 2015

I prefer if declare -F function 1>/dev/null, but we can also use if function

@rcaloras
Copy link
Owner

rcaloras commented Nov 1, 2015

Either one works for me. Feel free to submit another pull with that if you'd like.

@rcaloras
Copy link
Owner

rcaloras commented Nov 1, 2015

👍 LGTM 🍔 🍟

rcaloras added a commit that referenced this pull request Nov 1, 2015
…nction

test precmd_function without test compound command and command substitution
@rcaloras rcaloras merged commit f4a04f8 into rcaloras:master Nov 1, 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

2 participants