Skip to content

Remove all extra whitespace PSR2 does not like - #20391

Merged
ChristophWurst merged 6 commits into
masterfrom
refactor/spaces-cleanup
Apr 9, 2020
Merged

Remove all extra whitespace PSR2 does not like#20391
ChristophWurst merged 6 commits into
masterfrom
refactor/spaces-cleanup

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member

So, a bigger one this time, but I tried to keep everything separate in commits so it's reviewable in steps.

There is a number of rules for PSR2 that remove whitespace from various spots where it's not really expected. We're very close to PSR2 after this PR I think. But let's see 🙏

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@skjnldsv

Copy link
Copy Markdown
Member

Cheater!

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

quick scan seemed all good

@ChristophWurst
ChristophWurst merged commit 008e6d7 into masterApr 9, 2020
@ChristophWurst
ChristophWurst deleted the refactor/spaces-cleanup branch April 9, 2020 18:39
@rullzerrullzer mentioned this pull request Apr 10, 2020
59 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewstechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@ChristophWurst@skjnldsv@rullzer