Skip to content

Conversation

@rodrigoprimo
Copy link
Collaborator

While adding a new test to CapitalPDangitUnitTest.1.inc in an unrelated commit, I noticed unnecessary trailing spaces in this file. Instead of removing them just for this one test, I opted to create this PR removing unnecessary trailing spaces from all test files using Squiz.WhiteSpace.SuperfluousWhitespace. I'm including only the files where it was easy to check that the spaces were not added intentionally.

While adding a new test to `CapitalPDangitUnitTest.1.inc`, I noticed unnecessary trailing spaces in this file. Instead of removing them just for this one test, I opted to create this commit removing unnecessary trailing spaces from all test files using `Squiz.WhiteSpace.SuperfluousWhitespace`. I'm including only the files where it was easy to check that the spaces were not added intentionally.
Copy link
Member

@jrfnl jrfnl left a comment

Choose a reason for hiding this comment

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

Thanks for doing this clean-up @rodrigoprimo !

@jrfnl jrfnl added this to the 3.2.0 milestone Jul 18, 2025
@GaryJones GaryJones merged commit 911ebd7 into WordPress:develop Jul 18, 2025
41 checks passed
@rodrigoprimo rodrigoprimo deleted the remove-superfluous-whitespaces branch July 22, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants