Conversation
Bump composer.json for Laravel 13.7 toolchain. No source changes
required — df-azure's only Illuminate import is Support\Arr (stable
across L11 -> L13), and tests inherit from df-core / df-database /
df-file TestCase bases (already on shift/laravel-13).
require:
- php ^8.3
- laravel/helpers ^1.8 (polyfills array_get/camel_case/array_except)
require-dev:
- laravel/framework ^13.7
- phpunit/phpunit ^11.5.3
- orchestra/testbench ^11.0
- mockery/mockery ^1.6
- nunomaduro/collision ^8.6
Top-level "version": "0.19.99" pin satisfies the circular-dep
constraint chain (df-database ~1.4 -> df-file ~0.8 -> df-azure ~0.19)
during Stage 1 isolated install with sibling shift/laravel-13
path-repos.
Microsoft Azure SDK pins (azure-storage-{common,blob,table},
azure-documentdb-php-sdk) unchanged. SDKs are abandoned upstream
but install cleanly under PHP 8.3 (php >=5.6.0 constraint).
Validated:
- Stage 1 (isolated): composer install resolves, all 14 namespaced
classes autoload, helper polyfills active, Laravel 13.7.0
- Stage 2 (host-app shift-173254): df-azure registers via package
discovery, all 14 classes + Azure SDK BlobRestProxy/TableRestProxy
resolve under bootstrapped L13.7 app
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Composer-only bump to Laravel 13.7. df-azure has zero L13-blocking source patterns (no
DispatchesJobs, noCorsService, nogetDatesoverrides, nosetUpBeforeClasstypos, noIlluminate\Database\Connectionextensions) — its only Illuminate import isSupport\Arr, which is stable across L11 -> L13.Changes
composer.json:php ^8.3,laravel/helpers ^1.8(polyfillsarray_get/camel_case/array_exceptfor in-source legacy call sites in dependent packages).laravel/framework ^13.7,phpunit/phpunit ^11.5.3,orchestra/testbench ^11.0,mockery/mockery ^1.6,nunomaduro/collision ^8.6.0.19.99so Stage 1 can resolve the structural circular constraint chain (df-database ~1.4→df-file ~0.8→df-azure ~0.19) when sibling shift/laravel-13 branches are wired in via path repos.Microsoft Azure SDK pins (unchanged)
microsoft/azure-storage-common ~1.5.2microsoft/azure-storage-blob ~1.5.4microsoft/azure-storage-table ~1.1.6dreamfactory/azure-documentdb-php-sdk ~0.2.0These SDKs are abandoned upstream (latest releases Sep 2022) but install cleanly under PHP 8.3 — their constraint is
php >=5.6.0, no Laravel coupling. No newer L13/PHP-8.3-aware fork exists; if Microsoft ever ships a successor (e.g.microsoft/microsoft-azure-storage-blob), that's a separate migration outside the L13 upgrade scope.Source changes
None.
Test plan
composer installresolves; all 14 namespaced classes (ServiceProvider,Services\{Blob,Table,DocumentDB},Resources\{Table,DocumentDbTable},Models\{AzureConfig,TableConfig,DocumentDbConfig,BlobConfig},Components\{AzureBlobFileSystem,DocumentDBConnection},Database\Schema\{AzureTableSchema,DocumentDbSchema}) autoload; helper polyfills active;Application::VERSION = 13.7.0; lint-clean across allsrc/*.php.shift-173254): df-azure participates in package discovery, all 14 classes + Azure SDKBlobRestProxy/TableRestProxyresolve under bootstrapped L13.7 app.Notes
Pre-existing latent bug observed but not addressed (out of scope for L13 upgrade):
src/Components/AzureBlobFileSystem.phpandsrc/Resources/Table.phpimportMicrosoftAzure\Storage\Common\ServiceException, but the SDK's actual class isMicrosoftAzure\Storage\Common\Exceptions\ServiceException. Theinstanceofandcatchclauses against this aliased class never fire. Identical behavior on master (predates L13). File a follow-up issue.Part of the Laravel 11 -> 13 upgrade campaign (Wave 4).