Skip to content

Update redis to the latest version 🚀 #4087

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

Merged
merged 2 commits into from
Aug 23, 2017
Merged

Conversation

greenkeeper[bot]
Copy link
Contributor

@greenkeeper greenkeeper bot commented Aug 8, 2017

Version 2.8.0 of redis just got published.

Dependency redis
Current Version 2.7.1
Type dependency

The version 2.8.0 is not covered by your current version range.

Without accepting this pull request your project will work just like it did before. There might be a bunch of new features, fixes and perf improvements that the maintainers worked on for you though.

I recommend you look into these changes and try to get onto the latest version of redis.
Given that you have a decent test suite, a passing build is a strong indicator that you can take advantage of these changes by merging the proposed change into your project. Otherwise this branch is a great starting point for you to work on the update.


Commits

The new version differs by 35 commits.

  • 1380ad6 fix: test on old node versions
  • 6694c91 v.2.8.0
  • ad8355a fix: add command tests
  • 79558c5 doc: improve README readability by limiting chars to 80
  • 51fdbb7 chore: improve new add_command and add documentation
  • 0437aa4 enabled adding abritary commands
  • 4f7f1ad fix: silence auth errors on reconnect
  • 42e8bd6 chore: stop testing Node.js 7
  • 937081b fix: dependency
  • e5c8f81 fix: revert some dev dependency updates to run tests
  • 7e9dda1 chore: test node 8
  • 16632f4 fix emitting internal auth error on reconnect
  • 50774ae fix: accept UPPER_CASE commands in send_command
  • 789471b chore: update peer dependency version
  • 6934270 fix: always copy subscribe unsubscribe arguments

There are 35 commits in total.

See the full diff

Not sure how things should work exactly?

There is a collection of frequently asked questions and of course you may always ask my humans.


Your Greenkeeper Bot 🌴

@codecov
Copy link

codecov bot commented Aug 8, 2017

Codecov Report

Merging #4087 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4087      +/-   ##
==========================================
- Coverage   90.74%   90.72%   -0.02%     
==========================================
  Files         116      116              
  Lines        7916     7916              
==========================================
- Hits         7183     7182       -1     
- Misses        733      734       +1
Impacted Files Coverage Δ
src/Adapters/Auth/meetup.js 84.21% <0%> (-5.27%) ⬇️
...dapters/Storage/Postgres/PostgresStorageAdapter.js 96.38% <0%> (-0.13%) ⬇️
src/Adapters/Auth/instagram.js 94.11% <0%> (+5.88%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 47f00e6...1de2174. Read the comment docs.

@flovilmart flovilmart merged commit 3ed6dd2 into master Aug 23, 2017
@greenkeeper greenkeeper bot deleted the greenkeeper/redis-2.8.0 branch August 23, 2017 15:35
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.

1 participant