Closed Geod24 closed 8 months ago
✅ PR OK, no changes in deprecations or warnings
Total deprecations: 0
Total warnings: 0
Build statistics:
statistics (-before, +after)
-executable size=5286232 bin/dub
+executable size=5282136 bin/dub
rough build time=61s
@thewilsonator : This one too ? ;)
The PackageManager will always eagerly load subpackages, whether they are path based or embedded (see addPackages). So calling getInternalSubPackage if getSubPackage returned null will never yield a non-Nullable object. In short, we can simplify that code greatly, and as Package.getInternalSubPackage does not handle all use case and require using the PackageManager anyway, just deprecate it and direct people to the right function.