Skip to content

Enhancement: Enable no_mixed_echo_print fixer - #864

Merged
kamil-tekiela merged 2 commits into
php:masterfrom
localheinz:feature/no-mixed-echo-print
Dec 6, 2023
Merged

Enhancement: Enable no_mixed_echo_print fixer#864
kamil-tekiela merged 2 commits into
php:masterfrom
localheinz:feature/no-mixed-echo-print

Conversation

@localheinz

@localheinzlocalheinz commented Dec 6, 2023

Copy link
Copy Markdown
Contributor

This pull request

  • enables the no_mixed_echo_print fixer
  • runs make coding-standards

Follows #559.

💁‍♂️ For reference, see https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/blob/v3.40.2/doc/rules/alias/no_mixed_echo_print.rst.

@localheinz
localheinzforce-pushed the feature/no-mixed-echo-print branch from 1396d63 to 3f92052CompareDecember 6, 2023 14:14
Comment thread.php-cs-fixer.php
'line_ending' => true,
'new_with_parentheses' => true,
'no_extra_blank_lines' => true,
'no_mixed_echo_print' => true,

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Note that this fixer can be configured with the use option to prefer echo or print.

Using the default option (echo) applies fewer changes across the code base.

@kamil-tekiela
kamil-tekiela merged commit a528e6e into php:masterDec 6, 2023
@localheinz
localheinz deleted the feature/no-mixed-echo-print branch December 6, 2023 14:20
@localheinz

Copy link
Copy Markdown
ContributorAuthor

Thank you, @kamil-tekiela!

Sign up for freeto 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

@localheinz@kamil-tekiela