Skip to content

Cakephp5.3 deprecations#30

Merged
cnizzardini merged 6 commits into
mainfrom
cakephp5.3-deprecations
Feb 15, 2026
Merged

Cakephp5.3 deprecations#30
cnizzardini merged 6 commits into
mainfrom
cakephp5.3-deprecations

Conversation

@cnizzardini
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the codebase to address CakePHP 5.3 deprecations, primarily focusing on renaming the plugin class and updating CI configurations.

Changes:

  • Renamed the plugin class from Plugin to CakePreloaderPlugin to comply with CakePHP 5.3 naming conventions
  • Updated phpcs.xml to include vendor paths for coding standards
  • Modified CI workflow to test against PHP 8.5 and adjusted CakePHP version testing matrix

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/CakePreloaderPlugin.php Renamed plugin class from Plugin to CakePreloaderPlugin to address CakePHP 5.3 deprecation of generic plugin class names
phpcs.xml Updated installed_paths to use vendor directory paths for cakephp-codesniffer and slevomat coding standards
.github/workflows/ci.yml Updated PHP version matrix to test against 8.1 and 8.5; changed CakePHP version testing from '^5.1' to '^5.0'

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml Outdated
Comment thread phpcs.xml
cnizzardini and others added 3 commits February 14, 2026 13:41
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cnizzardini cnizzardini merged commit 97a21ff into main Feb 15, 2026
6 checks passed
@cnizzardini cnizzardini deleted the cakephp5.3-deprecations branch February 15, 2026 15:51
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