๐Ÿ‡ฎ๐Ÿ‡ณIndia @Shreyas gowda

Account created on 26 September 2023, about 1 year ago
#

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

patch #12 is applied successfully

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

resolved some of this error
still few warnings and errors are left

FILE: C:\wamp64\www\contribution\block_list_override\README.txt
----------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
----------------------------------------------------------------------
14 | WARNING | Line exceeds 80 characters; contains 108 characters
43 | WARNING | Line exceeds 80 characters; contains 303 characters
----------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\block_list_override\src\Controller\DefaultController.php
-----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------
14 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
-----------------------------------------------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\block_list_override\src\Form\SettingsForm.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 14 ERRORS AND 16 WARNINGS AFFECTING 16 LINES
-------------------------------------------------------------------------------------------------------------------------
13 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
62 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
62 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
74 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
74 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
87 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
87 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
92 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
92 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
99 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
99 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
102 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
102 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
108 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
108 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
127 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
127 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
143 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
143 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
150 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
150 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
164 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
164 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
174 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
174 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
181 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
181 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
188 | WARNING | Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
188 | ERROR | Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
199 | WARNING | Possible useless method overriding detected
-------------------------------------------------------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\block_list_override\tests\src\Functional\BlockListOverrideTest.php
---------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------
92 | WARNING | Line exceeds 80 characters; contains 82 characters
---------------------------------------------------------------------------------------------------

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

I applied the patch #10 and it fixes errors of the PHPCS but there's 1 warning remaining .

FILE: ...ules/contrib/jquery_ui/assets/vendor/jquery.ui/themes/base/progressbar.css
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------------

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

NO phpcs errors or warnings found

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

fixed all the phpcs issues one error and warning is left

FILE: C:\wamp64\www\contribution\preview_site\src\Deploy\DeployPluginBase.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
60 | ERROR | The trigger_error message 'The method
| | \Drupal\preview_site\Deploy\DeployPluginInterface::deployFilePath
| | is deprecated in Deploy plugins. It should be implemented.' does
| | not match the relaxed standard format: %thing% is deprecated in
| | %deprecation-version% any free text %removal-version%.
| | %extra-info%. See %cr-link%
--------------------------------------------------------------------------------

FILE: ...\wamp64\www\contribution\preview_site\src\Generate\TomeStaticExtension.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
102 | WARNING | Possible useless method overriding detected
--------------------------------------------------------------------------------

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

i found Zero errors but i can see few warnings

FILE: C:\wamp64\www\contribution\cookies\modules\cookies_filter\README.md
-------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------
25 | WARNING | Line exceeds 80 characters; contains 90 characters
-------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\cookies\modules\cookies_instagram\css\cookies_instagram.css
--------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\cookies\modules\cookies_recaptcha\css\cookies_recaptcha.css
--------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
--------------------------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\cookies\modules\cookies_twitter_media\css\cookies_twitter_media.css
----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
----------------------------------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\cookies\modules\cookies_video\css\cookies_media.css
------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
------------------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\cookies\modules\cookies_video\css\cookies_video_embed_field.css
------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
------------------------------------------------------------------------------------------------

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

ERRORS are fixed but still left with some warnings

FILE: ...p64\www\contribution\better_exposed_filters\better_exposed_filters.install
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------
223 | WARNING | Line exceeds 80 characters; contains 84 characters
227 | WARNING | Line exceeds 80 characters; contains 83 characters
--------------------------------------------------------------------------------

FILE: ...tribution\better_exposed_filters\includes\better_exposed_filters.theme.inc
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
162 | WARNING | '@todo' should match the format '@todo Fix problem X here.'
--------------------------------------------------------------------------------

FILE: C:\wamp64\www\contribution\better_exposed_filters\README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
42 | WARNING | Line exceeds 80 characters; contains 94 characters
----------------------------------------------------------------------

FILE: ...osed_filters\src\Plugin\better_exposed_filters\filter\FilterWidgetBase.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------------
116 | WARNING | Only string literals should be passed to t() where possible
185 | WARNING | Only string literals should be passed to t() where possible
251 | WARNING | Line exceeds 80 characters; contains 104 characters
260 | WARNING | Line exceeds 80 characters; contains 84 characters

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

Resolved the phpcs issues and commited it please review

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shreyas gowda

Shreyas gowda โ†’ made their first commit to this issueโ€™s fork.

Production build 0.71.5 2024