Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DX: Use Utils::naturalLanguageJoin() in implode calls #7032

Merged

Conversation

paulbalandan
Copy link
Contributor

Follow up to #7022

Changes mainly done on documentation generation and exception messages.

@paulbalandan paulbalandan force-pushed the use-utils-natural-language-join branch from 6e5939e to 29a9252 Compare June 6, 2023 11:37
Copy link
Member

@Wirone Wirone left a comment

Choose a reason for hiding this comment

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

Looks good in general, but I have one serious doubt 🙂.

src/Fixer/Operator/BinaryOperatorSpacesFixer.php Outdated Show resolved Hide resolved
@paulbalandan paulbalandan force-pushed the use-utils-natural-language-join branch from 32b2d32 to 62c9fa1 Compare June 7, 2023 03:50
@Wirone Wirone merged commit 3d2254d into PHP-CS-Fixer:master Jun 7, 2023
14 checks passed
@Wirone
Copy link
Member

Wirone commented Jun 7, 2023

Thanks @paulbalandan 🍻

@paulbalandan paulbalandan deleted the use-utils-natural-language-join branch June 7, 2023 08:11
niklam pushed a commit to niklam/PHP-CS-Fixer that referenced this pull request Jun 19, 2023
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.

None yet

2 participants