Fix the issues reported by phpcs

FILE: /var/www/html/web/modules/custom/address_display/tests/src/Kernel/Formatter/AddressDisplayTest.php ------------------------------------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ------------------------------------------------------------------------------------------------------------------------------------------------- 22 | ERROR | Do not disable strict config schema checking in tests. Instead ensure your module properly declares its schema for configurations. ------------------------------------------------------------------------------------------------------------------------------------------------- FILE: /var/www/html/web/modules/custom/address_display/README.txt ---------------------------------------------------------------------- FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE ---------------------------------------------------------------------- 12 | WARNING | Line exceeds 80 characters; contains 85 characters ---------------------------------------------------------------------- FILE: /var/www/html/web/modules/custom/address_display/address_display.info.yml ---------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE ---------------------------------------------------------------------------------------------- 7 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" ---------------------------------------------------------------------------------------------- FILE: /var/www/html/web/modules/custom/address_display/src/Plugin/Field/FieldFormatter/AddressDisplayFormatter.php ------------------------------------------------------------------------------------------------------------------ FOUND 2 ERRORS AFFECTING 2 LINES ------------------------------------------------------------------------------------------------------------------ 98 | ERROR | [ ] Description for the @return value is missing 101 | ERROR | [x] Opening brace should be on the same line as the declaration ------------------------------------------------------------------------------------------------------------------ PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY ------------------------------------------------------------------------------------------------------------------

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Issue created by @arunkumark
Assigned to annmarysruthy
Comment almost 2 years ago →
🇮🇳India annmarysruthy
@annmarysruthy opened merge request.
Issue was unassigned.
Status changed to Needs work almost 2 years ago9:49am 2 March 2023
Comment almost 2 years ago →
🇮🇳India annmarysruthy
Raised MR!3. Some of the phpcs issues are fixed. only 1 issue in /var/www/html/web/modules/custom/address_display/tests/src/Kernel/Formatter/AddressDisplayTest.php is pending.
Comment almost 2 years ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
Comment almost 2 years ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
- - address + - drupal:address
As already said in a MR comment, the Address module is not part of Drupal core. Its project name is not drupal.
Status changed to Needs review almost 2 years ago5:40pm 7 March 2023
Comment almost 2 years ago →
🇵🇱Poland Krzysztof Domański Poland
Addressed #12 📌 Fix the issues reported by phpcs Fixed 📌 Overridden test methods require void return type hints Fixed
Comment almost 2 years ago →
🇵🇱Poland lamp5 Rzeszow
This issue duplicated this 2 issues: 📌 Update .info.yml file Fixed and 📌 Replace README.txt to README.md and update README.md Fixed so only one thing to fix is opening brace.
Comment almost 2 years ago →
System Message

lamp5 → committed e47d7da5 on 8.x-1.x
Issue #3345418 by annmarysruthy, imustakim, lamp5, Krzysztof Domański:...
Status changed to Fixed almost 2 years ago12:39pm 8 March 2023
Comment almost 2 years ago →
🇵🇱Poland lamp5 Rzeszow
Comment almost 2 years ago →
System Message
lamp5 → closed merge request !3
Comment almost 2 years ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Fix the issues reported by phpcs

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments & Activities