Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Jan 6, 2026

Description

As per the inline comment replacing the section:

While PSR12.Operators.OperatorSpacing inherits the ignoreNewlines and ignoreSpacingBeforeAssignments properties
from the Squiz.WhiteSpace.OperatorSpacing sniff, these properties are not handled in the PSR12 sniff,
so are deliberately not mentioned in this documentation.

Related issues/external references

Discovered while reviewing upstream PR PHPCSStandards/PHP_CodeSniffer#1356

PR checklist

  • I have checked there is no other PR open for the same change.
  • I have read the Contribution Guidelines.
  • I grant the project the right to include my changes under the BSD-3-Clause license (and I have the right to grant these rights).
  • I have verified that my changes comply with the projects coding standards.
  • [When adding a new Wiki page] I have added the new page to the _Sidebar.md file.
  • [When adding/updating output examples] I have verified via the GitHub Actions artifact that the pre-processing handles the command correctly.

…tion

As per the inline comment replacing the section:

> While PSR12.Operators.OperatorSpacing inherits the `ignoreNewlines` and `ignoreSpacingBeforeAssignments` properties
> from the `Squiz.WhiteSpace.OperatorSpacing` sniff, these properties are not handled in the PSR12 sniff,
> so are deliberately not mentioned in this documentation.
@jrfnl jrfnl force-pushed the wiki/customisable-sniff-props-fix-psr12-operatorspacing branch from 9805f98 to 7616e14 Compare January 6, 2026 17:52
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

=== This is an auto-generated comment ===

Thank you for your PR.
A dry-run has been executed on your PR, executing all markdown pre-processing for the wiki files.

Please review the resulting final markdown files via the created artifact.
This is especially important when adding new pages or updating auto-generated output blocks.

N.B.: the above link will automatically be updated when this PR is updated.

@jrfnl jrfnl merged commit f91bb55 into main Jan 6, 2026
14 checks passed
@jrfnl jrfnl deleted the wiki/customisable-sniff-props-fix-psr12-operatorspacing branch January 6, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants