Skip to content

Update overall setup of the module#3

Merged
remyperona merged 12 commits intodevelopfrom
chore/setup
Nov 27, 2025
Merged

Update overall setup of the module#3
remyperona merged 12 commits intodevelopfrom
chore/setup

Conversation

@remyperona
Copy link
Copy Markdown
Contributor

@remyperona remyperona commented Nov 26, 2025

This PR modernizes the module setup by updating PHP and WordPress version requirements, reorganizing the project structure, and improving code quality tooling. The changes prepare the library for modern PHP development standards.

Key changes:

  • Bumps minimum PHP version from 7.0 to 7.4 and WordPress from 5.3 to 6.6
  • Reorganizes source code into inc/ directory with updated autoloading
  • Modernizes test infrastructure with proper PHPUnit configuration and test case naming conventions
  • Updates development dependencies and CI workflows

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 PR modernizes the module setup by updating PHP and WordPress version requirements, reorganizing the project structure, and improving code quality tooling. The changes prepare the library for modern PHP development standards.

Key changes:

  • Bumps minimum PHP version from 7.0 to 7.4 and WordPress from 5.3 to 6.6
  • Reorganizes source code into inc/ directory with updated autoloading
  • Modernizes test infrastructure with proper PHPUnit configuration and test case naming conventions
  • Updates development dependencies and CI workflows

Reviewed changes

Copilot reviewed 44 out of 57 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
phpstan.neon.dist Increased PHPStan level to 5, updated paths to scan inc/ and Tests/, added WordPress stub files
phpcs.xml.dist Updated scan paths to inc/ and Tests/, removed several rule exclusions, updated PHP/WP version requirements
composer.json Updated PHP requirement to 7.4+, modernized dev dependencies, restructured autoloading to use inc/ directory
inc/*.php Renamed parameter $default to $default_value for clarity, added void return types, fixed spacing
Tests/Unit/*.php Added strict types, renamed test classes to PascalCase convention, added PHPDoc type hints, removed extra blank lines
Tests/Integration/*.php Added strict types, renamed test classes to PascalCase convention, added PHPDoc type hints, fixed method names
Tests/Fixtures/*.php Aligned array formatting for consistency
.github/workflows/*.yml Replaced legacy workflow files with modern reusable workflows for PHPUnit, PHPStan, and PHPCS
Comments suppressed due to low confidence (2)

Tests/Integration/SiteOptions/DeleteTest.php:19

  • Corrected spelling of 'testShoulDeleteOption' to 'testShouldDeleteOption'.
    inc/OptionArray.php:1
  • [nitpick] The spacing in the PHPDoc is inconsistent. Line 60 has two spaces between @param mixed and $value, while line 61 has only one space between @param mixed and $default_value. These should be consistently aligned.

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

Comment thread composer.json Outdated
@remyperona remyperona merged commit 0695480 into develop Nov 27, 2025
3 checks passed
@remyperona remyperona deleted the chore/setup branch November 27, 2025 14:49
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