Closed phil-davis closed 1 year ago
Merging #213 (ecf945f) into master (5220b2b) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #213 +/- ##
=========================================
Coverage 94.21% 94.21%
Complexity 260 260
=========================================
Files 15 15
Lines 881 881
=========================================
Hits 830 830
Misses 51 51
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This should not break any consumer. For anyone already using v7, it will report some types more accurately and so their consumer-code will be better analyzed by phpstan.
Note: most consumers will currently be using
sabre/http
along withsabre/dav
and other sabre-io components. So the dependency requirements of the currentsabre/dav
4.4.0 will pull insabre/http
5.1.7 anyway. So there are probably not many projects usingsabre/http
major version 7 yet anyway.Eventually I will work through the type-declarations and phpstan level 9 throughout the sabre-io repos - I have some work in
vobject
that I need to get back to, and then try the mainsabre/dav
repo. And then we can do a set of major releases of all that...