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 an optional third parameter to redisget() to specify a default value #209

Merged
merged 1 commit into from
May 19, 2017

Conversation

petems
Copy link
Member

@petems petems commented May 19, 2017

Rebase of #207 with docs

@petems petems force-pushed the ghoneycutt-redisget_default branch 2 times, most recently from dd9e3d2 to 047cf1b Compare May 19, 2017 10:59
* Specifies a default value if the value is not
found or theres a failure to connect to redis
* Adds spec and acceptance tests for new default
value
@ghoneycutt
Copy link
Member

Thanks! In accordance with SemVer, could you please release a new version of this module? This contribution would move us from 3.0.0 to 3.1.0.

@petems
Copy link
Member Author

petems commented May 19, 2017

Yep yep, will do! 👍

@ghoneycutt
Copy link
Member

Thank you. Could you please post the release here, so I'm notified.

@petems
Copy link
Member Author

petems commented May 19, 2017

Can do, we're trying to get the auto-release stuff working so it's automatic in the future also 👍

@arioch
Copy link
Contributor

arioch commented May 19, 2017

@ghoneycutt: as requested.
The auto-release failed, manually uploaded to the forge in the meantime.

cegeka-jenkins pushed a commit to cegeka/puppet-redis that referenced this pull request Feb 16, 2021
* Specifies a default value if the value is not
found or theres a failure to connect to redis
* Adds spec and acceptance tests for new default
value
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.

3 participants