Add security supported versions to testing matrix

[s01ipsist]

The Github Actions testing matrix currently only runs against versions <=7.0.

As per https://endoflife.date/redis, the security supported versions are 7.2, 7.4 and 8.0.
8.2 also came out yesterday: https://github.com/redis/redis/releases

This leaves versions <=7.0 in the test matrix, even though they are no longer supported upstream.

[s01ipsist]

https://github.com/s01ipsist/redis-rb/actions/runs/16739960872/job/47386180705

Both 8.0 and 8.2 have a failure on the geopos function that appears to be around decimal precision.

  1) Failure:
TestCommandsGeo#test_geopos [test/redis/commands_on_geo_test.rb:64]:
--- expected
+++ actual
@@ -1 +1 @@
-[["15.08726745843887329", "37.50266842333162032"]]
+[["15.087267458438873", "37.50266842333162"]]


  2) Failure:
TestCommandsGeo#test_geopos_nonexistant_location [test/redis/commands_on_geo_test.rb:75]:
--- expected
+++ actual
@@ -1 +1 @@
-[nil, ["15.08726745843887329", "37.50266842333162032"]]
+[nil, ["15.087267458438873", "37.50266842333162"]]

Any preference on how to handle this?

[byroot]

Any preference on how to handle this?

You can check the server version:

if version >= "8.2"
  assert ...
else
  assert ...
end

Might be worth renaming version in server_version though.

[s01ipsist]

You can check the server version:

I've added conditional logic for the expected values for Palermo and Catania. Let me know if this works for you

[byroot]

Works for me. Please just clean your git history.

[s01ipsist]

Ok... Not quite clear on that request. Do you mean like what a squash merge would do?