Skip to content

Conversation

@justlevine
Copy link
Collaborator

What

This PR runs

  • npm run lint:css:fix
  • npm run lint:php:fix
  • npm run lint:js:fix

and commits the results.

Important

This PR is a follow up to #27 , which should be merged first.

Why

Related Issue(s):

How

Testing Instructions

Screenshots

Additional Info

Checklist

  • I have read the Contribution Guidelines.
  • I have read the Development Guidelines.
  • My code is tested to the best of my abilities.
  • My code passes all lints (ESLint etc.).
  • My code has detailed inline documentation.
  • I have updated the project documentation as needed.

Copilot AI review requested due to automatic review settings November 24, 2025 23:04
Copy link
Contributor

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 applies automated linting fixes across the codebase by running npm run lint:css:fix, npm run lint:php:fix, and npm run lint:js:fix. The changes primarily enforce coding standards including short array syntax, early exit patterns, static closures, and improved code organization.

Key changes:

  • Converted PHP array syntax from array() to [] throughout the codebase
  • Refactored control structures to use early exits for better readability
  • Updated webpack build paths from assets/build to build
  • Added comprehensive test infrastructure (PHPUnit, PHPStan configuration)
  • Updated project documentation and CI/CD workflows

Reviewed changes

Copilot reviewed 48 out of 57 changed files in this pull request and generated no comments.

Show a summary per file
File Description
webpack.config.js Simplified build configuration with updated output paths
uninstall.php Array syntax updates and early exit refactoring
tests/bootstrap.php New PHPUnit test bootstrap file
phpunit.xml.dist New PHPUnit configuration
phpstan.neon.dist New PHPStan static analysis configuration
phpcs.xml.dist Comprehensive PHPCS ruleset with multiple coding standards
package.json Major dependency updates and reorganized scripts
onedesign.php Restructured plugin initialization with autoloader
inc/traits/trait-singleton.php Modernized with static::class and array syntax
inc/classes/rest/*.php Array syntax and early exit pattern updates
inc/classes/*.php Consistent code style improvements
composer.json Enhanced configuration with additional dev dependencies
.github/workflows/*.yml New CI/CD workflows for testing and releases
Documentation files Updated setup instructions and contribution guidelines

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

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@justlevine justlevine requested a review from up1512001 November 24, 2025 23:07
@justlevine justlevine merged commit 5028fd9 into develop Nov 26, 2025
7 of 10 checks passed
@justlevine justlevine deleted the chore/autofix-lints branch November 26, 2025 12:31
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.

3 participants