Add node 18 to CI

[silverwind]

Let's see how this goes.

[TysonAndre]

I'm not familiar with this project's test strategy but this looks reasonable with node 18 LTS starting October 25th

Unrelatedly, I see redis 7.0 stable was recently released.

Also, a question for other maintainers: I've used include in other projects to reduce the number of jobs while allowing testing more combinations without a large number of excludes. Would it make sense to start testing more redis versions with this?

matrix:
  include:
    - node: 12.x
      redis: '2'
    - node: 12.x
      redis: '6'
    - node: 12.x
      redis: '7'
      # ...

[luin]

Also, a question for other maintainers: I've used include in other projects to reduce the number of jobs while allowing testing more combinations without a large number of excludes. Would it make sense to start testing more redis versions with this?

@TysonAndre You mean only test all Redis versions with only one Node.js version? That does make sense to me but I'm okay with 4 (Node.js versions) * 3 (Redis versions) so I think we can stay without using include.

[github-actions]

🎉 This PR is included in version 5.2.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀