Skip to content

PLT-951 Add PHP 8 support#158

Draft
mrliptontea wants to merge 12 commits intoPLT-951-modernise-codefrom
PLT-951-add-php-8-support
Draft

PLT-951 Add PHP 8 support#158
mrliptontea wants to merge 12 commits intoPLT-951-modernise-codefrom
PLT-951-add-php-8-support

Conversation

@mrliptontea
Copy link
Copy Markdown
Member

@mrliptontea mrliptontea commented Mar 19, 2026

  • PLT-951
  • Add PHP 8 support
  • Require ext-mongodb >= 1.7.0
    • Enforce that $readPreference is a string
    • Switch from ReadPreference->getMode() to getModeString()
  • Require psr/log instead of Monolog
    • Monolog is now a dev-only package
  • Add Resque compatibility layer
    • Allow library to work against current latest release but also development which includes namespace changes and also requires that job classes extend Resque\Job\Job.
  • Decouple Tripod\Mongo\Jobs\JobBase from Tripod\Mongo\DriverBase
    • Extract LoggerTrait from DriverBase
    • Few methods from DriverBase used by jobs were re-implemented.
    • Make Tripod\Mongo\Jobs\JobBase extend Resque\Job\Job.
  • Bump arc2 library
    • Allow versions 2 & 3.

@mrliptontea mrliptontea force-pushed the PLT-951-add-php-8-support branch from 0f63c9b to 0ada92c Compare March 19, 2026 15:05
Copy link
Copy Markdown

@mike-dean-talis mike-dean-talis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to an untrained tripod-ist.

@mrliptontea mrliptontea force-pushed the PLT-951-add-php-8-support branch from 0ada92c to 28d6ed1 Compare March 24, 2026 12:18
@mrliptontea mrliptontea force-pushed the PLT-951-add-php-8-support branch from 28d6ed1 to 20cf02f Compare March 24, 2026 12:42
@mrliptontea mrliptontea force-pushed the PLT-951-add-php-8-support branch from 20cf02f to 9611893 Compare March 24, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants