Skip to content

Conversation

@sreichel
Copy link
Contributor

@sreichel sreichel commented Nov 1, 2025

Copilot AI review requested due to automatic review settings November 1, 2025 06:42
@sreichel sreichel added the chore label Nov 1, 2025
@github-actions github-actions bot added Component: Core Relates to Mage_Core Component: Adminhtml Relates to Mage_Adminhtml Component: ConfigurableSwatches Relates to Mage_ConfigurableSwatches php-cs-fixer labels Nov 1, 2025
@sreichel sreichel merged commit 95acf1b into OpenMage:main Nov 1, 2025
5 checks passed
@sreichel sreichel deleted the cleanup/phpdoc_single_line_var_spacing branch November 1, 2025 06:42
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 pull request adds the phpdoc_single_line_var_spacing rule to the PHP-CS-Fixer configuration and applies it across the codebase to standardize spacing in single-line @var PHPDoc comments.

  • Adds phpdoc_single_line_var_spacing rule to the PHP-CS-Fixer configuration
  • Removes trailing spaces after type declarations in single-line @var PHPDoc comments across four core files

Reviewed Changes

Copilot reviewed 1 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.php-cs-fixer.dist.php Adds the phpdoc_single_line_var_spacing rule to enforce proper spacing in single-line @var PHPDoc comments
app/code/core/Mage/Core/Model/Email/Template/Filter.php Removes trailing spaces from two @var PHPDoc comments
app/code/core/Mage/ConfigurableSwatches/Helper/Mediafallback.php Removes trailing space from @var PHPDoc comment
app/code/core/Mage/Adminhtml/Block/Report/Product/Lowstock/Grid.php Removes trailing space from @var PHPDoc comment
app/code/core/Mage/Adminhtml/Block/Promo/Quote/Grid.php Removes trailing space from @var PHPDoc comment

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 1, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Component: Adminhtml Relates to Mage_Adminhtml Component: ConfigurableSwatches Relates to Mage_ConfigurableSwatches Component: Core Relates to Mage_Core php-cs-fixer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant