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 support for POSIX default values #248

Merged
merged 1 commit into from Apr 16, 2020

Conversation

@bbc2
Copy link
Collaborator

@bbc2 bbc2 commented Apr 14, 2020

When interpolation is active, variable expansion can use a default value, like in Bash.

When interpolation is active, variable expansion can use a default
value, like in Bash.
@bbc2 bbc2 self-assigned this Apr 14, 2020
@coveralls
Copy link

@coveralls coveralls commented Apr 14, 2020

Coverage Status

Coverage increased (+0.05%) to 89.806% when pulling eaefca8 on bbc2:default-in-variable-expansion into b491bbc on theskumar:master.

@thaJeztah
Copy link

@thaJeztah thaJeztah commented Apr 15, 2020

Thanks @bbc2!

@ulyssessouza ptal

@ulyssessouza
Copy link
Contributor

@ulyssessouza ulyssessouza commented Apr 16, 2020

LGTM!

@bbc2 bbc2 merged commit c389b45 into theskumar:master Apr 16, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.05%) to 89.806%
Details
@bbc2 bbc2 deleted the bbc2:default-in-variable-expansion branch Apr 16, 2020
@ulyssessouza
Copy link
Contributor

@ulyssessouza ulyssessouza commented Apr 19, 2020

@bbc2 Thanks a lot for that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants