Skip to content

Reformat code base to PSR2#2862

Closed
dvikan wants to merge 2 commits intoRSS-Bridge:masterfrom
dvikan:reformat-psr2
Closed

Reformat code base to PSR2#2862
dvikan wants to merge 2 commits intoRSS-Bridge:masterfrom
dvikan:reformat-psr2

Conversation

@dvikan
Copy link
Contributor

@dvikan dvikan commented Jun 24, 2022

This change was automatically performed by phpcs:

$ ./vendor/bin/phpcbf --standard=PSR2 actions/ bridges/ caches/ contrib/ formats/ index.php lib/ tests/

$ ./vendor/bin/phpcbf --version
PHP_CodeSniffer version 3.6.2 (stable) by Squiz (http://www.squiz.net)

Increased the line limit length because some of the produced code exceeded it. Excluded InstructablesBridge.php because there is currently a problem with the prtester. I used a dummy author in the commit.

I hope someone can double check the process to make sure there are no undesired changes. Maybe @jtojnar? Or @f0086?

Checkout the current master, do the transform and then diff with this PR:

git remote add dvikan git@github.com:dvikan/rss-bridge
git fetch dvikan reformat-psr2
git checkout bridges/InstructablesBridge.php
git diff dvikan/reformat-psr2

We can do PSR12 at a later time. I also suggest later that we convert to shorthand arrays and enforce it.

dvikan and others added 2 commits June 24, 2022 18:47
This change was automatically performed by phpcs:

$ ./vendor/bin/phpcbf --standard=PSR2 actions/ bridges/ caches/ contrib/ formats/ index.php lib/ tests/

$ ./vendor/bin/phpcbf --version
PHP_CodeSniffer version 3.6.2 (stable) by Squiz (http://www.squiz.net)
@dvikan dvikan mentioned this pull request Jun 30, 2022
@dvikan dvikan closed this Jun 30, 2022
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.

2 participants