-
Notifications
You must be signed in to change notification settings - Fork 22
Upgrade dependencies #61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
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
This ensures that the encrypter is only created at the point of it being used.
tommitchelmore
requested changes
Oct 15, 2025
Collaborator
tommitchelmore
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments
This was referenced Oct 15, 2025
tommitchelmore
requested changes
Oct 15, 2025
tommitchelmore
approved these changes
Oct 15, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Dependency Upgrades
Upgrades major version of most packages. This will form the basis of an 8.0 release.
Breaking Changes
PHP - 8.1 - 8.4
Support for PHP < 8.1 has been removed.
Laminas Zend Framework Bridge removed
Any imports that start with
Zend\Diactorosmust be re-written toLaminas\DiactorosFacades -
blast/facadesremovedBlast facades was 10 years out of date, and has been removed. We've decided to take ownership of this code.
All facades will need to be modified as follows:
Internally, any calls to
\Blast\Facades\FacadeFactoryshould be replaced by\Rareloop\Lumberjack\FacadeManagerFuture work on this looks like adding Laravel-style mocking features.
Encryption
Brought a forked version of Dcrypt into the core that supports PHP 8.4. This requires a namespace update if you are using this anywhere:
Log levels
We now use
Monolog\Levelobjects throughout Lumberjack. Any instances ofMonolog\Loggerneed to be replaced.For example:
'logs' => [ 'enabled' => true, 'path' => false, - 'level' => Monolog\Logger::ERROR, + 'level' => Monolog\Level::Error, ],Non-breaking changes
Ignition
We've moved from
symfony/debugtospatie/ignition- mirroring #56. This addresses deprecation warnings in #52.Composer
To support
composer update --prefer-lowestwe had to set minimum versions on multiple sub-dependencies as composer was bringing in versions that do not support PHP 8.4. These are:require:require-dev:+"antecedent/patchwork": "^2.2.3"