fix: matched offsetSet and offsetUnset signature

[Mondotosz]

added function signature to Repository offsetSet and offsetUnset methods to match the signature of ArrayAccess.

I'm using the library for my first real Laravel project and those two methods keep on sending deprecation warnings.
I still haven't gone too deep in php OOP but I believe adding the same signature as found on php.net wouldn't break existing codebases.

I tried it on my project by modifying the code in the vendor directory and didn't get any deprecation warnings or exceptions.

Hope it helps and thank you for the library!

[codecov-commenter]

Codecov Report

Merging #30 (3d413c2) into master (cfd7008) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master       #30   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        43        43           
===========================================
  Files              5         5           
  Lines            100       100           
===========================================
  Hits             100       100           

[veelasky]

@Mondotosz thank you for your contribution..

however, you should also disable the test on php7.4 and drop support for php7.4 for this to be able to qualify for merging.

[Mondotosz]

@Mondotosz thank you for your contribution..

however, you should also disable the test on php7.4 and drop support for php7.4 for this to be able to qualify for merging.

I didn't realize it was unsupported by php7.4. Thank you for taking the time to update the pipeline accordingly.
Is it really alright to drop support for php7.4?

[veelasky]

I didn't realize it was unsupported by php7.4. Thank you for taking the time to update the pipeline accordingly. Is it really alright to drop support for php7.4?

it is time to move on, whoever run 7.4 can always restraint to the old version.