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

Expand elasticache parameters #118

Merged
merged 2 commits into from
Nov 7, 2016
Merged

Expand elasticache parameters #118

merged 2 commits into from
Nov 7, 2016

Conversation

tjcorr
Copy link
Collaborator

@tjcorr tjcorr commented Nov 4, 2016

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.1%) to 51.793% when pulling 524ed81 on expand-ElastiCache into 8dec124 on master.

@bkrodgers
Copy link
Contributor

Any concerns about breaking changes here? These do look like changes we should make, so I don't see a problem as long as we bump the version number accordingly for the breaking change.

@tjcorr
Copy link
Collaborator Author

tjcorr commented Nov 7, 2016

None of this code was in a release yet so no worries about breaking changes. See original PR: #111

@bkrodgers bkrodgers merged commit 007ce1d into master Nov 7, 2016
@bkrodgers bkrodgers deleted the expand-ElastiCache branch November 7, 2016 22:43
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

3 participants