Fix the errors/warnings reported by PHP_CodeSniffer

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/media_entity_unsplash/README.md ----------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES ----------------------------------------------------------------------------------------------------------------- 6 | WARNING | Line exceeds 80 characters; contains 82 characters 40 | WARNING | Line exceeds 80 characters; contains 99 characters 45 | WARNING | Line exceeds 80 characters; contains 127 characters 51 | WARNING | Line exceeds 80 characters; contains 81 characters 68 | WARNING | Line exceeds 80 characters; contains 82 characters ----------------------------------------------------------------------------------------------------------------- FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/media_entity_unsplash/src/Plugin/media/Source/Unsplash.php ------------------------------------------------------------------------------------------------------------------------------------------------ FOUND 2 ERRORS AFFECTING 2 LINES ------------------------------------------------------------------------------------------------------------------------------------------------ 8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\Display\EntityViewDisplayInterface. 25 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph ------------------------------------------------------------------------------------------------------------------------------------------------ PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY ------------------------------------------------------------------------------------------------------------------------------------------------ FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/media_entity_unsplash/src/UnsplashFetcherInterface.php ---------------------------------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE ---------------------------------------------------------------------------------------------------------------------------------------- 26 | WARNING | Line exceeds 80 characters; contains 84 characters ---------------------------------------------------------------------------------------------------------------------------------------- FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/media_entity_unsplash/src/UnsplashFetcher.php ------------------------------------------------------------------------------------------------------------------------------- FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES ------------------------------------------------------------------------------------------------------------------------------- 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Unsplash\HttpClient. 59 | ERROR | [x] Each PHP statement must be on a line by itself 95 | WARNING | [ ] Line exceeds 80 characters; contains 156 characters ------------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY ------------------------------------------------------------------------------------------------------------------------------- Time: 179ms; Memory: 10MB

Comments & Activities

Issue created by @paraderojether
Merge request !53469572: Fix the issues reported by phpcs → (Merged) created by paraderojether
Pipeline finished with Failed
6 months ago
Total: 171s
#260907
Issue was unassigned.
Status changed to Needs review 6 months ago1:50am 22 August 2024
Comment 6 months ago →
🇵🇭Philippines paraderojether
I created an MR to fix the issues reported by phpcs, please review.

Thank you.
First commit to issue fork.
Status changed to Needs work 5 months ago9:26am 30 August 2024
Comment 5 months ago →
🇳🇱Netherlands batigolix Utrecht
There are still things phpcs and other things to fix. See the pipeline
Status changed to Fixed 5 months ago10:14am 31 August 2024
Comment 5 months ago →
🇳🇱Netherlands batigolix Utrecht
The remaining fixes can be done at a later moment as they break D9 compatibility.
Pipeline finished with Skipped
5 months ago
#270194
Comment 5 months ago →
System Message

batigolix → committed f7832c38 on 1.0.x authored by paraderojether →
Issue #3469572 by paraderojether, batigolix: Fix the errors/warnings...
Comment 5 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Fix the errors/warnings reported by PHP_CodeSniffer

Merge Requests

!5Fix the errors/warnings reported by PHP_CodeSniffer
Merged

Comments & Activities

Fix the errors/warnings reported by PHP_CodeSniffer

Merge Requests

!5Fix the errors/warnings reported by PHP_CodeSnifferMerged

Comments & Activities

!5Fix the errors/warnings reported by PHP_CodeSniffer
Merged