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

Remove function keyword from standup-helper.sh #26

Merged
merged 1 commit into from
Apr 20, 2017

Conversation

jackmransom
Copy link
Contributor

@jackmransom jackmransom commented Apr 20, 2017

function as far as I can tell is an optional, non-standard keyword in bash and on some platforms the script fails with a syntax error if the keyword is included (I've personally found this to be the case on my machine running Debian. Which I assume is defaulting to running the script via sh which doesn't support function)

Simply removing function appears to fix issue #14 on the machines I've tested this fix on.

@notwaldorf notwaldorf merged commit 1a21e3e into notwaldorf:master Apr 20, 2017
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