feat: use php7.4 as minimum

[DeepDiver1975]

refs #1415

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9fb8b91) 97.32% compared to head (9296cf2) 97.22%.
Report is 4 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1502      +/-   ##
============================================
- Coverage     97.32%   97.22%   -0.11%     
+ Complexity     2833     2831       -2     
============================================
  Files           175      175              
  Lines          9425     9010     -415     
============================================
- Hits           9173     8760     -413     
+ Misses          252      250       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[phil-davis]

@DeepDiver1975 there were 1807 unit tests running - for exampe https://github.com/sabre-io/dav/actions/runs/6862503131/job/18660341581
But only 887 unit tests have run in gthis PR: https://github.com/sabre-io/dav/actions/runs/6856309480/job/18643234604?pr=1502

Maybe renaming various test file to TestAbstract.php has caused that?

[DeepDiver1975]

I will look into this.

[DeepDiver1975]

@phil-davis we have now 1640 tests. Previously test within sabre/http were executed as well - but they no longer exist in the composer package. This is fine form my pov as it is no the job of sabre/dav to run sabre/http tests

[phil-davis]

Previously test within sabre/http were executed as well
'
I had wondered why that was done, but without knowing the history I didn't like to stop running those sabre/http tests.

[phil-davis]

LGTM

[phil-davis]

I will merge and we can progress from here towards a major version with type declarations etc.