🇵🇭Philippines @paraderojether

Account created on 4 November 2022, about 2 years ago
#

Merge Requests

More

Recent comments

🇵🇭Philippines paraderojether

Hi

I reviewed MR!8 and confirmed all coding standard issues are fixed and it passed on the Gitlab CI

I also checked PHP_CodeSniffer and no phpcs issues found.

➜  simple_wysiwyg git:(1.0.x) curl https://git.drupalcode.org/project/simple_wysiwyg/-/merge_requests/8.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 12342    0 12342    0     0  41947      0 --:--:-- --:--:-- --:--:-- 43153
patching file .cspell-project-words.txt
patching file .gitlab-ci.yml
patching file 'css/simple-wysiwyg.css'
patching file 'js/simple-wysiwyg.js'
➜  simple_wysiwyg git:(1.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig simple_wysiwyg
➜  contrib git:(main) ✗

Moving this to RTBC!
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!6, applied it against Upload size per role 1.0.x-dev and confirmed it fixes the issues reported by PHP_CodeSniffer.

 upload_size_per_role git:(1.0.x) curl https://git.drupalcode.org/project/upload_size_per_role/-/merge_requests/6.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  8787    0  8787    0     0  29328      0 --:--:-- --:--:-- --:--:-- 29989
patching file .cspell-project-words.txt
patching file .gitlab-ci.yml
patching file 'src/Form/UploadSizePerRoleSettingsForm.php'
patching file upload_size_per_role.module
➜  upload_size_per_role git:(1.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig upload_size_per_role
➜  contrib git:(main) ✗

phpcs, phpstan, and cspell also passed on the Gitlab CI

Moving this to RTBC!
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!8, applied it against the Database logging ban operation 2.0.x and confirmed it fixes the issues reported by PHP_CodeSniffer.

dblog_ban git:(2.0.x) curl https://git.drupalcode.org/project/dblog_ban/-/merge_requests/8.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5137    0  5137    0     0  12003      0 --:--:-- --:--:-- --:--:-- 12201
patching file CHANGELOG.txt
patching file README.md
patching file 'src/Controller/IpBanUnbanController.php'
patching file 'src/Form/BanUnbanConfirmFormBase.php'
patching file 'tests/modules/dblog_ban_test/dblog_ban_test.info.yml'
patching file 'tests/src/Unit/RouteProcessorCsrfAjaxTest.php'
➜  dblog_ban git:(2.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig dblog_ban
➜  contrib git:(main) ✗

However, it display phpcs issues on the Gitlab CI

Kindly check and advise, please.
Thank you.

🇵🇭Philippines paraderojether

Hi alok_singh,

I reviewed MR!7, applied it against Bootstrap PointNet and it did not apply successfully, I got the error below:

error: patch failed: css/bootstrap_pointnet_style.css:3
error: css/bootstrap_pointnet_style.css: patch does not apply

I did not change the issue status so other contributors can also review it.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!4, applied it against the helpfulness 2.x-dev, and confirmed it fixes the issues reported by phpcs.

helpfulness git:(2.x) curl https://git.drupalcode.org/project/helpfulness/-/merge_requests/4.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2611    0  2611    0     0    775      0 --:--:--  0:00:03 --:--:--   778
patching file 'src/Form/HelpfulnessBlockForm.php'
patching file 'src/Form/HelpfulnessConfirmDeleteFeedbackForm.php'
patching file 'src/Form/HelpfulnessReportForm.php'
patching file 'src/Plugin/Block/HelpfulnessBlock.php'
➜  helpfulness git:(2.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig helpfulness
➜  contrib git:(main) ✗

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!9, applied it against Holy - atomic theme 3.0.0-alpha1, and confirmed the "Advanced search" button is now fixed.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!10, applied it against Holy - atomic theme 3.0.x-dev, and confirmed the extra spacing on the Add new comment section is removed.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed patch #2 and confirmed it is applied successfully and it fixes the rendering issue of the table.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed patch #10 and confirmed it is applied successfully the close button is added when the search is open. I tested the close button on desktop and touch devices and no issues was found while testing.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!18, applied it against Facets Pretty Paths 2.0.x-dev, and confirmed the configure link is added on the module listing page.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi,

I reviewed MR!5, applied it against PCR 2.0.x-dev and confirmed it fixes the issues reported by phpcs.

 contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig pcr

FILE: /Users/jetherparadero/Q4/q4contributions/web/modules/contrib/pcr/pcr.module
------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------------------
  1 | ERROR   | [x] Missing file doc comment
 11 | ERROR   | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Routing\RouteMatchInterface.
 44 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/Q4/q4contributions/web/modules/contrib/pcr/src/Plugin/better_exposed_filters/filter/PrettyCheckboxesRadios.php
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\better_exposed_filters\Plugin\better_exposed_filters\filter\RadioButtons.
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Time: 338ms; Memory: 10MB

➜  contrib git:(main) ✗ cd pcr
➜  pcr git:(2.0.x) curl https://git.drupalcode.org/project/pcr/-/merge_requests/5.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1436    0  1436    0     0   3388      0 --:--:-- --:--:-- --:--:--  3443
patching file pcr.module
patching file 'src/Plugin/better_exposed_filters/filter/PrettyCheckboxesRadios.php'
➜  pcr git:(2.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig pcr
➜  contrib git:(main) ✗

It is also passed on the on the Gitlab CI https://git.drupalcode.org/issue/pcr-3474014/-/jobs/2734627.

Thank you.

🇵🇭Philippines paraderojether

Hi alyaj2a,

I reviewed MR!3, applied against Year Only 9.x-dev and there are still error show below:

➜  contrib ✗ cd yearonly
➜  yearonly git:(9.x) curl https://git.drupalcode.org/project/yearonly/-/merge_requests/13.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2441    0  2441    0     0   5693      0 --:--:-- --:--:-- --:--:--  5784
patching file 'src/Feeds/Target/YearOnly.php'
patching file 'src/Plugin/Field/FieldWidget/YearOnlyDefaultWidget.php'
➜  yearonly git:(9.x) ✗ cd ..
➜  contrib ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig yearonly

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/yearonly/src/Plugin/Field/FieldWidget/YearOnlyDefaultWidget.php
-------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------------
 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Field\WidgetBase.
-------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/yearonly/src/Plugin/Field/FieldType/YearOnlyItem.php
-------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Field\FieldStorageDefinitionInterface.
-------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------

Please check and advise, the issue exists on 9.x-dev.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!570, applied it against Project Browser 2.0.x-dev, and confirmed the cursor is changed to a pointer when hovering over the results tab.

I added a recorded video for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi mohammad-fayoumi,

I was not able to reproduce the issue, my Drupal version is 10.3.2 with PHP version 8.3.6, can you please specify in what Drupal core version the issue occurred?

Thank you.

🇵🇭Philippines paraderojether

Hi dharmeshbarot89,

I reviewed MR!22, and there is still 1 warning remaining shown below:

➜  view_password git:(6.0.x) curl https://git.drupalcode.org/project/view_password/-/merge_requests/22.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  7774    0  7774    0     0   4335      0 --:--:--  0:00:01 --:--:--  4350
patching file CHANGELOG.txt
patching file README.md
patching file 'config/install/view_password.settings.yml'
patching file 'config/schema/view_password.schema.yml'
patching file 'src/Form/PasswordSettingsForm.php'
patching file view_password.libraries.yml
patching file view_password.module
➜  view_password git:(6.0.x) ✗ cd ..
➜  contrib git:(1.0.0) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig view_password

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/view_password/src/Form/PasswordSettingsForm.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------
 45 | WARNING | Unused variable $trailing_slash_message.
---------------------------------------------------------------------------------------------------------------------------------

Time: 116ms; Memory: 10MB

Thank you.

🇵🇭Philippines paraderojether

Hi,

I reviewed MR!7, applied against Single User Role 8.x-1.x-dev and confirmed phpcs issues are now resolved.

PHP_CodeSniffer

➜  single_user_role git:(8.x-1.0-beta2) curl https://git.drupalcode.org/project/single_user_role/-/merge_requests/7.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4773    0  4773    0     0   2846      0 --:--:--  0:00:01 --:--:--  2859
patching file .gitlab-ci.yml
patching file single_user_role.module
patching file 'src/Form/SingleUserRoleConfigForm.php'
➜  single_user_role git:(8.x-1.0-beta2) ✗ cd ..
➜  contrib git:(1.0.0) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig single_user_role
➜  contrib git:(1.0.0) ✗

It also passed on the Gitlab CI: https://git.drupalcode.org/issue/single_user_role-3452991/-/jobs/2681621

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!8, applied against Micro Site 8.x-1.x-dev, and confirmed the error when opening the media library when adding a new micro site is fixed.

I added screenshots and a video for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi alyaj2a,

I reviewed MR!11, applied it against it Views Load More 2.x-dev, and there is still 1 error remaining after applying the MR.

views_load_more git:(2.x) curl https://git.drupalcode.org/project/views_load_more/-/merge_requests/11.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 20900    0 20900    0     0  21990      0 --:--:-- --:--:-- --:--:-- 22116
patching file README.md
patching file 'src/Ajax/VLMAppendCommand.php'
patching file 'src/EventSubscriber/VLMEventSubscriber.php'
patching file 'src/Plugin/views/pager/LoadMore.php'
patching file views_load_more.module
➜  views_load_more git:(2.x) ✗ cd ..
➜  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig views_load_more

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/views_load_more/src/EventSubscriber/VLMEventSubscriber.php
--------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\views\Ajax\ViewAjaxResponse.
--------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------------------

Time: 143ms; Memory: 10MB

Please check.
Thank you.

🇵🇭Philippines paraderojether

Hi lhridley,

I reviewed MR!3, applied it against Lunr Search 8.x-1.x-dev, and confirmed the configure link is added on the module listing page successfully.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi f0ns,

I reviewed patch #5, applied it against Layout disable 2.0.3, and confirmed the "navigation_layout" can't be disabled anymore.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi v.dovhaliuk,

I've tried to reproduce the issue but am unable to reproduce it.
Please provide the detailed Steps to Reproduce the issue.

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!6, applied it against Bootstrap DateTime Picker 4.0.0-beta3, and confirmed I can select Sunday as a date now.

I added a before and after screen recording for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi @alok_singh,

I tested MR!18 and confirmed it fixes the advanced search input fields overlapping issue on mobile devices.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

I created an MR to fix the issues reported by phpcs, please review.

Thank you.

🇵🇭Philippines paraderojether

Fix the issues reported by phpcs, please review.

Thank you.

🇵🇭Philippines paraderojether

I fixed the issues reported by phpcs, please review.

Thank you.

🇵🇭Philippines paraderojether

I fixed the remaining issue, please review.

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!4, applied against Media Thumbnails 8.x-1.x-dev and there is one phpcs error shown below:

✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig media_thumbnails

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/media_thumbnails/src/Plugin/MediaThumbnailManager.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------
 8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Cache\CacheBackendInterface.
---------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------

Time: 350ms; Memory: 10MB

Thank you.

🇵🇭Philippines paraderojether

I created MR to fix the remaining issues, please review.

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!33, applied against the Purge 8.x-3.x-dev and confirmed the configure link is added on the Purge UI module.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!6, applied it against Menu Admin per Menu 8.x-1.x-dev and there are still remaining phpcs issue shown below:

➜  menu_admin_per_menu git:(98d34cb) curl https://git.drupalcode.org/project/menu_admin_per_menu/-/merge_requests/16.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4928    0  4928    0     0   9338      0 --:--:-- --:--:-- --:--:--  9476
patching file README.md
patching file 'src/Routing/RouteSubscriber.php'
patching file 'tests/src/Kernel/MenuAdminPerMenuAccessTest.php'
patching file 'tests/src/Kernel/MenuAdminPerMenuPermissionsTest.php'
➜  menu_admin_per_menu git:(98d34cb) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig menu_admin_per_menu

FILE: /Users/jetherparadero/q3do/q3issues/web/modules/contrib/menu_admin_per_menu/src/Access/MenuAdminPerMenuAccess.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------
 26 | WARNING | Menu::loadMultiple calls should be avoided in classes, use dependency injection instead
 35 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-----------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/q3do/q3issues/web/modules/contrib/menu_admin_per_menu/src/MenuAdminPerMenuPermissions.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\StringTranslation\StringTranslationTrait.
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/q3do/q3issues/web/modules/contrib/menu_admin_per_menu/src/MenuAdminPerMenuAccessInterface.php
----------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Menu\MenuLinkInterface.
----------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------

Time: 548ms; Memory: 16MB

Kindly check and advise, please.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!2, applied it against Pager 3.0.0, and confirmed it fixes the issue reported by phcs.

➜  pager git:(main) ✗ curl https://git.drupalcode.org/project/pager/-/merge_requests/2.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3554    0  3554    0     0  11521      0 --:--:-- --:--:-- --:--:-- 11729
patching file 'src/PagerStorage.php'
patching file 'src/Plugin/Block/PagerBlock.php'
➜  pager git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig pager
➜  contrib git:(main) ✗

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!6, applied against Recently Read 8.x-1.x-dev, and confirmed it fixes the issues reported by phpcs.

➜  recently_read git:(2eb445a) curl https://git.drupalcode.org/project/recently_read/-/merge_requests/6.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 37599    0 37599    0     0  87599      0 --:--:-- --:--:-- --:--:-- 89521
patching file README.txt
patching file recently_read.info.yml
patching file recently_read.install
patching file recently_read.module
patching file recently_read.views.inc
patching file 'src/Entity/RecentlyRead.php'
patching file 'src/Entity/RecentlyReadInterface.php'
patching file 'src/Entity/RecentlyReadType.php'
patching file 'src/Entity/RecentlyReadTypeInterface.php'
patching file 'src/Form/RecentlyReadConfigForm.php'
patching file 'src/Form/RecentlyReadTypeForm.php'
patching file 'src/Plugin/views/relationship/RecentlyReadRelationship.php'
patching file 'src/RecentlyReadHtmlRouteProvider.php'
patching file 'src/RecentlyReadService.php'
patching file 'src/RecentlyReadServiceInterface.php'
patching file 'src/RecentlyReadStorageSchema.php'
patching file 'src/RecentlyReadTypeListBuilder.php'
patching file 'tests/src/Functional/RecentlyReadBlockTest.php'
➜  recently_read git:(2eb445a) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig recently_read
➜  contrib git:(main) ✗

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!5, and applied it against Systems Tags 2.0.1 and it is not applied cleanly.

➜  system_tags git:(2.0.1) curl https://git.drupalcode.org/project/system_tags/-/merge_requests/5.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 19046    0 19046    0     0  59017      0 --:--:-- --:--:-- --:--:-- 61637
Checking patch README.md...
Checking patch modules/theme/system_tags_theme.module...
error: while searching for:
 * Implements hook_theme_suggestions_HOOK_alter().
 */
function system_tags_theme_theme_suggestions_node_alter(array &$suggestions, array $variables) {
  $suggestions = array_merge($suggestions, _system_tag_node_suggestions('node__system_tag__', $variables['elements']['#node']));
}

/**

error: patch failed: modules/theme/system_tags_theme.module:27
error: modules/theme/system_tags_theme.module: patch does not apply
Checking patch src/Annotation/SystemTagFinder.php...
Checking patch src/Config/SystemPageConfigOverrider.php...
Checking patch src/Config/SystemTagDefinitions.php...
Checking patch src/Entity/SystemTag.php...
Checking patch src/Entity/SystemTagInterface.php...
Checking patch src/Form/SystemTagDeleteForm.php...
Checking patch src/Form/SystemTagForm.php...
Checking patch src/Plugin/Condition/SystemTags.php...
Checking patch src/Plugin/SystemTagFinder/SystemTagBlockFinder.php...
Checking patch src/Plugin/SystemTagFinder/SystemTagNodeFinder.php...
Checking patch src/SystemTagAccessControlHandler.php...
Checking patch src/SystemTagFinder/SystemTagFinderInterface.php...
Checking patch src/SystemTagFinder/SystemTagFinderManager.php...
Checking patch src/SystemTagFinder/SystemTagFinderManagerInterface.php...
Checking patch src/SystemTagFinder/SystemTagFinderPluginBase.php...
error: while searching for:
      $storage = $this->entityTypeManager->getStorage($entityTypeId);

      $query = $storage->getQuery()
        ->sort('changed', 'DESC', $langcode);
      $orCondition = $query->orConditionGroup();
      foreach ($fields as $field) {

error: patch failed: src/SystemTagFinder/SystemTagFinderPluginBase.php:95
error: src/SystemTagFinder/SystemTagFinderPluginBase.php: patch does not apply
Checking patch src/SystemTagHelper.php...
Checking patch src/SystemTagHelperInterface.php...
Checking patch src/SystemTagHtmlRouteProvider.php...
Checking patch src/SystemTagListBuilder.php...
Checking patch src/TwigExtension/SystemTagsTwigExtension.php...
error: while searching for:
namespace Drupal\system_tags\TwigExtension;

use Drupal\system_tags\SystemTagFinder\SystemTagFinderManagerInterface;

/**
 * Class SystemTagsTwigExtension.
 */
class SystemTagsTwigExtension extends \Twig_Extension {

  /**
   * The system tag finder manager.

error: patch failed: src/TwigExtension/SystemTagsTwigExtension.php:3
error: src/TwigExtension/SystemTagsTwigExtension.php: patch does not apply
Checking patch system_tags.module...
Checking patch system_tags.tokens.inc...
Checking patch tests/src/Functional/DefaultSystemTagsTest.php...

Kindly check and advise, please.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!1, applied it against Youtube Integration 1.0.1 and confirmed it fixes the issues reported by phpcs.

➜  youtubeintegrate git:(main) ✗ curl https://git.drupalcode.org/project/youtubeintegrate/-/merge_requests/1.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 47286    0 47286    0     0  85015      0 --:--:-- --:--:-- --:--:-- 86131
patching file 'css/youtube.css'
patching file 'src/Controller/YouTubeIntegrationController.php'
patching file 'src/Entity/YouTubePlaylistEntity.php'
patching file 'src/YouTubeAPI.php'
patching file 'templates/youtube-integration-playlist-item.html.twig'
patching file 'templates/youtube-integration-playlists.html.twig'
patching file youtube_integration.links.menu.yml
patching file youtube_integration.module
patching file youtube_integration.routing.yml
➜  youtubeintegrate git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig youtubeintegrate
➜  contrib git:(main) ✗

Moving this to RTBC.
Thank you.

🇵🇭Philippines paraderojether

Hi LOBsTerr,

I reviewed MR!25, applied it against CAS 2.1.0, and confirmed Cas setting is now translatable.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

I fix the remaining issues, please review.

Thank you.

🇵🇭Philippines paraderojether

paraderojether made their first commit to this issue’s fork.

🇵🇭Philippines paraderojether

I fix the remaining issue, please review.

Thank you.

🇵🇭Philippines paraderojether

paraderojether made their first commit to this issue’s fork.

🇵🇭Philippines paraderojether

Hi

I fix the remaining issues, please review.

Thank you

🇵🇭Philippines paraderojether

paraderojether made their first commit to this issue’s fork.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!7, applied it against United Nations Maps & Geoservices 1.2.0, and confirmed it fixes the issues reported by phpcs.

➜  unep_maps git:(main) ✗ curl https://git.drupalcode.org/project/unep_maps/-/merge_requests/7.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 11473    0 11473    0     0  27730      0 --:--:-- --:--:-- --:--:-- 28120
patching file README.md
patching file 'css/edw_mapbox.css'
patching file edw_maps.api.php
patching file edw_maps.module
patching file 'src/Controller/EdwMapsController.php'
patching file 'src/EdwMapsElementsTrait.php'
patching file 'src/Plugin/views/style/MapboxMapStyle.php'
patching file 'src/Services/EdwMapsDataService.php'
➜  unep_maps git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,info,txt,md,yml,twig unep_maps
➜  contrib git:(main) ✗

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!8, applied it against Event Scheduler 2.3.0-rc3 and it confirmed the configuration was added successfully.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!50, applied it against Layout Builder + and it confirmed the configuration was added successfully.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed patch #2, applied it against OPENAI Search 2.0.1 and it confirmed the configuration was added successfully.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!19, applied against Watchdog Mailer and it confirmed the configuration is added successfully.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

I updated the MR to fix the remaining issue, please review.

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!4 and there are still remaining issue shown below:

contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig shy_one_time

FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/shy_one_time/README.md
--------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------------
  99 | WARNING | Line exceeds 80 characters; contains 94 characters
 100 | WARNING | Line exceeds 80 characters; contains 115 characters
--------------------------------------------------------------------------------------

Time: 128ms; Memory: 10MB

.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!9, and there are still phpcs issues shown below:

contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig views_dependent_filters

FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/views_dependent_filters/README.txt
--------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------
 3 | WARNING | Line exceeds 80 characters; contains 106 characters
--------------------------------------------------------------------------------------------------


FILE: ...sers/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/views_dependent_filters/src/Plugin/views/filter/ViewsDependentFilter.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 11 WARNINGS AFFECTING 11 LINES
--------------------------------------------------------------------------------------------------------------------------------------
 130 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 143 | WARNING | Unused variable $form_state_copy.
 162 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 163 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 171 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 174 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 177 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 193 | WARNING | Unused variable $controller_filter.
 233 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 236 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
 253 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t()
     |         | instead
--------------------------------------------------------------------------------------------------------------------------------------

Time: 155ms; Memory: 12MB
🇵🇭Philippines paraderojether

Hi
I reviewed MR!7 and there are still remaining issues reported by phpcs show below:

contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig contentimport

FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/contentimport/contentimport.module
--------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 1 WARNING AFFECTING 3 LINES
--------------------------------------------------------------------------------------------------
 312 | ERROR   | [x] Whitespace found at end of line
 315 | ERROR   | [x] Expected 1 space before "?"; 0 found
 315 | ERROR   | [x] Expected 1 space before ":"; 0 found
 317 | WARNING | [x] A comma should follow the last multiline array item. Found: $path_data
--------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------

Time: 296ms; Memory: 14MB
🇵🇭Philippines paraderojether

Hi

I reviewed MR!17, confirmed it is applied cleanly and the configure link is added successfully.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!10, and confirmed it fixes the issues reported by phpcs.

tmgmt_openai curl https://git.drupalcode.org/project/tmgmt_openai/-/merge_requests/10.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  7849    0  7849    0     0  24386      0 --:--:-- --:--:-- --:--:-- 24917
patching file README.md
patching file 'src/OpenAiTranslatorUi.php'
patching file 'src/Plugin/tmgmt/Translator/OpenAiTranslator.php'
patching file tmgmt_openai.info.yml
patching file tmgmt_openai.install
➜  tmgmt_openai cd ..
➜  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig tmgmt_openai
➜  contrib

Moving this to RBTC.
Thank you.

🇵🇭Philippines paraderojether

I reviewed patch #10 and confirmed it fixes the issues reported by phpcs.

permission_watchdog git:(2.x) curl https://git.drupalcode.org/project/permission_watchdog/-/merge_requests/1.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  6652    0  6652    0     0  22518      0 --:--:-- --:--:-- --:--:-- 22937
patching file permission_watchdog.install
patching file permission_watchdog.module
patching file 'src/Entity/RoleChangeLog.php'
patching file 'src/Form/SettingsForm.php'
patching file 'src/Plugin/DevelGenerate/RoleChangeDevelGenerate.php'
patching file 'src/Plugin/Field/FieldFormatter/PermissionActionDefaultFormatter.php'
patching file 'src/Plugin/Field/FieldType/PermissionActionItem.php'
patching file 'tests/src/Functional/PermissionWatchdogTest.php'
➜  permission_watchdog git:(2.x) ✗ cd ..
➜  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig permission_watchdog
➜  contrib 

I also created an MR so that it would be easy for the maintainers to review.
Please review the MR!. Thank you.

🇵🇭Philippines paraderojether

paraderojether made their first commit to this issue’s fork.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!32, confirmed it is applied successfully and it fixes the duplicated allowed types help text issue.

I added screenshots for reference.
Moving this RTBC.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!2, and confirmed phpcs issues listed on the description are fixed, however there are still phpcs issues remaining after running this command: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig upload_node_pictures

contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig upload_node_pictures

FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/upload_node_pictures/README.md
----------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 24 WARNINGS AFFECTING 24 LINES
----------------------------------------------------------------------------------------------
  1 | WARNING | [ ] Line exceeds 80 characters; contains 234 characters
  3 | WARNING | [ ] Line exceeds 80 characters; contains 117 characters
  5 | WARNING | [ ] Line exceeds 80 characters; contains 121 characters
  7 | WARNING | [ ] Line exceeds 80 characters; contains 127 characters
  9 | WARNING | [ ] Line exceeds 80 characters; contains 107 characters
 13 | WARNING | [ ] Line exceeds 80 characters; contains 109 characters
 14 | WARNING | [ ] Line exceeds 80 characters; contains 153 characters
 15 | WARNING | [ ] Line exceeds 80 characters; contains 128 characters
 16 | WARNING | [ ] Line exceeds 80 characters; contains 106 characters
 17 | WARNING | [ ] Line exceeds 80 characters; contains 105 characters
 18 | WARNING | [ ] Line exceeds 80 characters; contains 111 characters
 19 | WARNING | [ ] Line exceeds 80 characters; contains 86 characters
 20 | WARNING | [ ] Line exceeds 80 characters; contains 94 characters
 24 | WARNING | [ ] Line exceeds 80 characters; contains 135 characters
 28 | WARNING | [ ] Line exceeds 80 characters; contains 128 characters
 31 | WARNING | [ ] Line exceeds 80 characters; contains 132 characters
 34 | WARNING | [ ] Line exceeds 80 characters; contains 211 characters
 37 | WARNING | [ ] Line exceeds 80 characters; contains 221 characters
 40 | WARNING | [ ] Line exceeds 80 characters; contains 148 characters
 43 | WARNING | [ ] Line exceeds 80 characters; contains 211 characters
 46 | WARNING | [ ] Line exceeds 80 characters; contains 167 characters
 49 | WARNING | [ ] Line exceeds 80 characters; contains 169 characters
 52 | WARNING | [ ] Line exceeds 80 characters; contains 181 characters
 55 | WARNING | [ ] Line exceeds 80 characters; contains 215 characters
 55 | ERROR   | [x] Expected 1 newline at end of file; 0 found
----------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------

Time: 218ms; Memory: 12MB

Moving this needs work.

🇵🇭Philippines paraderojether

I updated the MR to fix the remaining phpcs issues, please review.

Thank you.

🇵🇭Philippines paraderojether

paraderojether made their first commit to this issue’s fork.

🇵🇭Philippines paraderojether

I created MR to fix remaining issue, please review.

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed patch #2, and there is 1 phpcs remaining.

phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig entity_pager

FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/entity_pager/src/EntityPager.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\ContentEntityInterface.
-----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------

Time: 372ms; Memory: 12MB
🇵🇭Philippines paraderojether

Hi

I reviewed patch #1, and there are still phpcs issues remaining shown below:

codemirror_editor git:(147c3b7) curl https://www.drupal.org/files/issues/2023-07-31/CodeMirrorEditorWidget.php-3370177-14.patch | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   663  100   663    0     0    360      0  0:00:01  0:00:01 --:--:--   361
patching file 'src/Plugin/Field/FieldWidget/CodeMirrorEditorWidget.php'
➜  codemirror_editor git:(147c3b7) ✗ cd ..
➜  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig codemirror_editor

FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/codemirror_editor/codemirror_editor.module
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 6 ERRORS AFFECTING 4 LINES
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 62 | ERROR | The text '@deprecated Use Drupal\codemirror_editor\CodemirrorModeManagerInterface::getDefinitions() instead.' does not match the standard format: @deprecated in
    |       | %deprecation-version% and is removed from %removal-version%. %extra-info%.
 62 | ERROR | Each @deprecated tag must have a @see tag immediately following it
 67 | ERROR | The trigger_error message 'codemirror_editor_load_modes() is deprecated. Use Drupal\codemirror_editor\CodemirrorModeManagerInterface::getDefinitions() instead'
    |       | does not match the strict standard format: %thing% is deprecated in %deprecation-version% and is removed from %removal-version%. %extra-info%. See %cr-link%
 80 | ERROR | The text '@deprecated Use Drupal\codemirror_editor\CodemirrorModeManagerInterface::normalizeMode() instead.' does not match the standard format: @deprecated in
    |       | %deprecation-version% and is removed from %removal-version%. %extra-info%.
 80 | ERROR | Each @deprecated tag must have a @see tag immediately following it
 85 | ERROR | The trigger_error message 'codemirror_editor_load_modes() is deprecated. Use Drupal\codemirror_editor\CodemirrorModeManagerInterface::normalizeMode() instead' does
    |       | not match the strict standard format: %thing% is deprecated in %deprecation-version% and is removed from %removal-version%. %extra-info%. See %cr-link%
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/codemirror_editor/src/CodeMirrorLibraryBuilder.php
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 8 | ERROR | The text '@deprecated Use \Drupal\codemirror_editor\LibraryBuilder' does not match the standard format: @deprecated in %deprecation-version% and is removed from
   |       | %removal-version%. %extra-info%.
 8 | ERROR | Each @deprecated tag must have a @see tag immediately following it
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Time: 610ms; Memory: 12MB

Thank you.

🇵🇭Philippines paraderojether

paraderojether made their first commit to this issue’s fork.

🇵🇭Philippines paraderojether

Hello a.aaronjake, I can't replicate the issue on my end.

🇵🇭Philippines paraderojether

paraderojether made their first commit to this issue’s fork.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!5, and confirmed it fixes all the issues reported by phpcs.

auto_username git:(8.x-1.5) curl https://git.drupalcode.org/project/auto_username/-/merge_requests/5.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 25823    0 25823    0     0  67039      0 --:--:-- --:--:-- --:--:-- 68496
patching file .gitlab-ci.yml
patching file auto_username.module
patching file auto_username.services.yml
patching file 'config/install/auto_username.settings.yml'
patching file 'src/AutoUsernameOptions.php'
patching file 'src/AutoUsernamePunctuationOptions.php'
patching file 'src/AutoUsernameUtilities.php'
patching file 'src/Form/AutoUsernameSettingsForm.php'
➜  auto_username git:(8.x-1.5) ✗ cd ..
➜  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig auto_username
➜  contrib

Moving this to RTBC.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed Patch 3, applied against Bootstrap External Link Pop-up 2.x-dev and confirmed it fixes the issues reported by phpcs.

bootstrap_external_link_popup git:(2.x) curl https://www.drupal.org/files/issues/2023-05-04/fix-for-issues-reported-by-phpcs-3358093-3.patch | patch
 -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   736  100   736    0     0   1452      0 --:--:-- --:--:-- --:--:--  1474
patching file bootstrap_external_link_popup.module
➜  bootstrap_external_link_popup git:(2.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig bootstrap_external_link_popup
➜  contrib git:(main) ✗ 

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!8, applied against Migrate Generator 8.x-2.x-dev, and confirmed fixes the issues reported by phpcs.

migrate_generator git:(8.x-2.2) curl https://git.drupalcode.org/project/migrate_generator/-/merge_requests/8.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 33546    0 33546    0     0  78327      0 --:--:-- --:--:-- --:--:-- 79681
patching file README.md
patching file migrate_generator.post_update.php
patching file 'migrate_generator_export/README.md'
patching file 'migrate_generator_export/migrate_generator_export.links.task.yml'
patching file 'migrate_generator_export/src/Commands/DrushCommands.php'
patching file 'migrate_generator_export/src/ExportManager.php'
patching file 'migrate_generator_export/src/FieldsExportManager.php'
patching file 'migrate_generator_export/src/Form/EntityTypeExportConfigForm.php'
patching file 'migrate_generator_export/src/Form/ExportForm.php'
patching file 'migrate_generator_export/src/GeneratorExportPluginManager.php'
patching file 'migrate_generator_export/src/Plugin/GeneratorExportPluginDateBase.php'
patching file 'migrate_generator_export/src/Plugin/GeneratorExportPluginInterface.php'
patching file 'migrate_generator_export/src/Plugin/migrate_generator_export/process/FileExport.php'
patching file 'src/Commands/DrushCommands.php'
patching file 'src/GeneratorProcessPluginManager.php'
patching file 'src/Plugin/GeneratorProcessPluginBase.php'
patching file 'src/Plugin/GeneratorProcessPluginDateBase.php'
patching file 'src/Scanner.php'
➜  migrate_generator git:(8.x-2.2) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig migrate_generator
➜  contrib git:(main) ✗ 

Thank you.

🇵🇭Philippines paraderojether

I fix the remaining issues, please review.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!18, it is applied cleanly however there are still remaining phpcs 2 errors shown below:

convert_bundles git:(2.0.x) curl https://git.drupalcode.org/project/convert_bundles/-/merge_requests/18.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 37895    0 37895    0     0  76538      0 --:--:-- --:--:-- --:--:-- 77336
Checking patch convert_bundles.api.php...
Checking patch convert_bundles.install...
Checking patch convert_bundles.module...
Checking patch src/Controller/EntityController.php...
Checking patch src/ConvertBundles.php...
Checking patch src/Form/ConvertBundlesForm.php...
Checking patch src/Plugin/Action/ConvertBundlesActionBase.php...
Checking patch src/Plugin/Derivative/ConvertBundlesLocalTask.php...
Checking patch src/Routing/RouteSubscriber.php...
Applied patch convert_bundles.api.php cleanly.
Applied patch convert_bundles.install cleanly.
Applied patch convert_bundles.module cleanly.
Applied patch src/Controller/EntityController.php cleanly.
Applied patch src/ConvertBundles.php cleanly.
Applied patch src/Form/ConvertBundlesForm.php cleanly.
Applied patch src/Plugin/Action/ConvertBundlesActionBase.php cleanly.
Applied patch src/Plugin/Derivative/ConvertBundlesLocalTask.php cleanly.
Applied patch src/Routing/RouteSubscriber.php cleanly.
➜  convert_bundles git:(2.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig convert_bundles

FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/convert_bundles/convert_bundles.module
-----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------
 9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Database\Database.
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/convert_bundles/src/Form/ConvertBundlesForm.php
------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------
 14 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Routing\RouteBuilderInterface.
------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------

Time: 375ms; Memory: 12MB

Please check and advise.
Thank you.

🇵🇭Philippines paraderojether

Hi Chandansha,

I reviewed Patch #2, applied it against Search Tracking 1.0.0, and confirmed the confirmed the configure link in the module list is added successfully.

Applying Patch:

search_tracking git:(main) ✗ curl https://www.drupal.org/files/issues/2024-02-22/3423112-2.patch | patch -p1   
\  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   416  100   416    0     0   2263      0 --:--:-- --:--:-- --:--:--  2377
patching file search_tracking.info.yml
➜  search_tracking git:(main) ✗

I also added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi silvi.addweb,

I reviewed MR!2, applied it against Instagram Feeds Listing 1.0.0 and confirmed it fixes the issues reported by phpcs.

instagram_feeds_listing git:(main) ✗ curl https://git.drupalcode.org/project/instagram_feeds_listing/-/merge_requests/2.diff | patch -p1   
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  6820    0  6820    0     0  13750      0 --:--:-- --:--:-- --:--:-- 14004
patching file 'src/Form/InstagramFeedAdminForm.php'
patching file 'src/InstagramBasicDisplayApi.php'
➜  instagram_feeds_listing git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig instagram_feeds_listing
➜  contrib git:(main) ✗ 

Thank you.

🇵🇭Philippines paraderojether

Hi silvi.addweb,

I reviewed MR! 3, and the MR is skipped or not applied successfully, please see below:

instagram_feeds_listing git:(main) ✗ curl https://git.drupalcode.org/project/instagram_feeds_listing/-/merge_requests/3.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   593    0   593    0     0    927      0 --:--:-- --:--:-- --:--:--   938
Skipped patch 'instagram_feeds_listing.info.yml'.
➜  instagram_feeds_listing git:(main) ✗

Please check and advise.
Thanks.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!1, it is applied cleanly however there are phpcs issues shown below:

FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/timetrack/src/Form/FavoriteProjectForm.php
----------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------
 9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\timetrack\Services\ProjectService.
----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/timetrack/src/Form/ProjectForm.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------
 9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\EntityTypeManagerInterface.
---------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/timetrack/src/Form/TimeTrackingForm.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------
 8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Routing\RouteMatchInterface.
---------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/timetrack/src/Services/ProjectService.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------
 8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Component\Datetime\TimeInterface.
---------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/timetrack/src/Services/TimeEntryService.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------
 6 | WARNING | [x] Unused use statement
 7 | ERROR   | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Component\Datetime\TimeInterface.
-----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------

Time: 420ms; Memory: 12MB
🇵🇭Philippines paraderojether

Hi sakthi_dev

I reviewed MR!10, however it the MR is skipped or not applied cleanly.

smallbusiness git:(main) ✗ curl https://git.drupalcode.org/project/smallbusiness/-/merge_requests/10.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   623    0   623    0     0   1232      0 --:--:-- --:--:-- --:--:--  1248
Skipped patch 'smallbusiness.theme'.
smallbusiness git:(main) ✗

Please check and advise.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!12, applied against Ridiculously Responsive Social Sharing Buttons 8.x-2.x-dev and there are still phpcs issues shown below:

jetherparadero@PrometSources-MacBook-Pro-3 contrib % phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig rrssb

FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/rrssb/rrssb.module
-------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------
 20 | WARNING | Global constants should not be used, move it to a class or interface
-------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/rrssb/src/Form/RRSSBSettingsForm.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------------------------
  79 | WARNING | Variable $require_follow inside unset call is undefined.
 186 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 189 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/rrssb/src/Plugin/Block/RRSSBBlock.php
-------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------
 27 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-------------------------------------------------------------------------------------------------------

Time: 310ms; Memory: 12MB

Kindly check and advise, please.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!6, applied against Simple Style Guide 2.0x-dev, confirmed patch applied cleanly however there are still remaining phpcs issue shown below:

jetherparadero@PrometSources-MacBook-Pro-3 contrib % phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig simple_styleguide

FILE: /Users/jetherparadero/d_o/drupal-issues/web/modules/contrib/simple_styleguide/src/Tests/StyleguidePageTest.php
---------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------------------------------------
 20 | ERROR   | Do not disable strict config schema checking in tests. Instead ensure your module properly declares its schema for configurations.
 70 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
---------------------------------------------------------------------------------------------------------------------------------------------------

Time: 215ms; Memory: 10MB

jetherparadero@PrometSources-MacBook-Pro-3 contrib %

Please check and advise.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed patch 23, applied against Diff 8.x-1.x-dev, and confirmed it fixes the issues reported by phpcs.

jetherparadero@PrometSources-MacBook-Pro-3 diff % curl https://www.drupal.org/files/issues/2024-04-29/3013942-coding-standard-23.patch | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 19618  100 19618    0     0  55959      0 --:--:-- --:--:-- --:--:-- 57029
patching file README.md
patching file 'css/diff.double-column.css'
patching file 'css/diff.single-column.css'
patching file diff.links.menu.yml
patching file diff.module
patching file diff.services.yml
patching file 'src/DiffEntityParser.php'
patching file 'src/DiffLayoutBase.php'
patching file 'src/Form/GeneralSettingsForm.php'
patching file 'src/Form/RevisionOverviewForm.php'
patching file 'src/Plugin/diff/Field/EntityReferenceFieldBuilder.php'
patching file 'src/Plugin/views/field/DiffFrom.php'
patching file 'src/Plugin/views/field/DiffTo.php'
patching file 'tests/modules/diff_test/diff_test.module'
patching file 'tests/src/Functional/DiffPluginEntityTest.php'
patching file 'tests/src/Functional/DiffPluginTest.php'
patching file 'tests/src/Functional/DiffRevisionTest.php'
jetherparadero@PrometSources-MacBook-Pro-3 diff % cd ..
jetherparadero@PrometSources-MacBook-Pro-3 contrib % phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig diff
jetherparadero@PrometSources-MacBook-Pro-3 contrib %

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!5, and confirmed it is applied however I got this warning warning: 1 line adds whitespace errors. while applying the patch.

jetherparadero@PrometSources-MacBook-Pro-3 button_field % curl https://git.drupalcode.org/project/button_field/-/merge_requests/5.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 22752    0 22752    0     0  49076      0 --:--:-- --:--:-- --:--:-- 49785
<stdin>:11: trailing whitespace.
- Button field reload action has been removed. Use the Rules Core
Checking patch README.txt...
Checking patch button_field.module...
Checking patch src/Plugin/Field/FieldFormatter/ButtonFieldBase.php...
Checking patch src/Plugin/Field/FieldFormatter/ButtonFieldHtml.php...
Checking patch src/Plugin/Field/FieldFormatter/ButtonFieldImage.php...
Checking patch src/Plugin/Field/FieldType/ButtonField.php...
Checking patch src/Plugin/Field/FieldWidget/ButtonFieldBase.php...
Checking patch src/Plugin/Field/FieldWidget/ButtonFieldHtml.php...
Checking patch src/Plugin/Field/FieldWidget/ButtonFieldImage.php...
Checking patch src/Plugin/migrate/field/d7/ButtonField.php...
Checking patch tests/src/TestSite/ButtonFieldInstallTestScript.php...
Applied patch README.txt cleanly.
Applied patch button_field.module cleanly.
Applied patch src/Plugin/Field/FieldFormatter/ButtonFieldBase.php cleanly.
Applied patch src/Plugin/Field/FieldFormatter/ButtonFieldHtml.php cleanly.
Applied patch src/Plugin/Field/FieldFormatter/ButtonFieldImage.php cleanly.
Applied patch src/Plugin/Field/FieldType/ButtonField.php cleanly.
Applied patch src/Plugin/Field/FieldWidget/ButtonFieldBase.php cleanly.
Applied patch src/Plugin/Field/FieldWidget/ButtonFieldHtml.php cleanly.
Applied patch src/Plugin/Field/FieldWidget/ButtonFieldImage.php cleanly.
Applied patch src/Plugin/migrate/field/d7/ButtonField.php cleanly.
Applied patch tests/src/TestSite/ButtonFieldInstallTestScript.php cleanly.
warning: 1 line adds whitespace errors.
jetherparadero@PrometSources-MacBook-Pro-3 button_field % cd ..
jetherparadero@PrometSources-MacBook-Pro-3 contrib % phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml button_field
jetherparadero@PrometSources-MacBook-Pro-3 contrib % phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig button_field
jetherparadero@PrometSources-MacBook-Pro-3 contrib %

Please check and advise for the warning after applying patch.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!3, applied against VariationCache 8.x-1.5 and confirmed the patch is applied cleanly and it fixes the issues reported by phpcs.

jetherparadero@PrometSources-MacBook-Pro-3 variationcache % curl https://git.drupalcode.org/project/variationcache/-/merge_requests/3.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  6710    0  6710    0     0   6932      0 --:--:-- --:--:-- --:--:--  6982
patching file README.md
patching file 'src/Cache/CacheRedirect.php'
patching file 'src/Cache/VariationCache.php'
patching file 'src/Cache/VariationCacheFactory.php'
patching file 'src/Cache/VariationCacheFactoryInterface.php'
patching file 'src/Cache/VariationCacheInterface.php'
patching file variationcache.module
jetherparadero@PrometSources-MacBook-Pro-3 variationcache % cd ..
jetherparadero@PrometSources-MacBook-Pro-3 contrib % phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml variationcache
jetherparadero@PrometSources-MacBook-Pro-3 contrib %

+1 RTBC
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!3, applied against Mega Slider 1.5.4, confirmed the patch applied however there are still phpcs that are not fixed.

Kindly check and advise, please.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!1, applied against Session Invalidator 1.00, and confirmed it is applied successfully and the hook help is added.

I added screenshots for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi govind_giri_goswami,

I reviewed MR!1, and it is applied successfully but the content is not related to the module and it is not helpful for the users.

Kindly check and advise, please.
Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed MR!1, applied it against Media Entity PodToo 1.x-dev, confirmed patch was applied successfully, and fixed all the issues reported by phpcs.

jetherparadero@PrometSources-MacBook-Pro podtoo % curl https://git.drupalcode.org/project/podtoo/-/merge_requests/1.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3204    0  3204    0     0   4666      0 --:--:-- --:--:-- --:--:--  4711
Checking patch src/Form/PodTooForm.php...
Checking patch src/Plugin/Field/FieldFormatter/PodTooEmbedFormatter.php...
Checking patch tests/src/Functional/PodTooEmbedFormatterTest.php...
Applied patch src/Form/PodTooForm.php cleanly.
Applied patch src/Plugin/Field/FieldFormatter/PodTooEmbedFormatter.php cleanly.
Applied patch tests/src/Functional/PodTooEmbedFormatterTest.php cleanly.
jetherparadero@PrometSources-MacBook-Pro podtoo % cd ..
jetherparadero@PrometSources-MacBook-Pro contrib % phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml podtoo
jetherparadero@PrometSources-MacBook-Pro contrib %

Thank you.

🇵🇭Philippines paraderojether

Hi

I reviewed patch 2, applied it against CloudFilt - Prevent & block bad bots - Bot Manager 1.0.x-dev, confirmed patch was applied successfully, and fixed all the issues reported by phpcs.

command used: phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml cloudfilt

jetherparadero@PrometSources-MacBook-Pro cloudfilt % curl https://www.drupal.org/files/issues/2024-03-13/Issues-3427593-Fixed-phpcs-issues.patch | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5618  100  5618    0     0  11082      0 --:--:-- --:--:-- --:--:-- 11349
Checking patch cloudfilt.info.yml...
Checking patch cloudfilt.module...
Checking patch src/Form/CloudfiltConfigForm.php...
Checking patch src/StackMiddleware/CloudfiltStackMiddleware.php...
Applied patch cloudfilt.info.yml cleanly.
Applied patch cloudfilt.module cleanly.
Applied patch src/Form/CloudfiltConfigForm.php cleanly.
Applied patch src/StackMiddleware/CloudfiltStackMiddleware.php cleanly.
jetherparadero@PrometSources-MacBook-Pro cloudfilt % cd ..
jetherparadero@PrometSources-MacBook-Pro contrib % phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml cloudfilt
jetherparadero@PrometSources-MacBook-Pro contrib %

Thank you.

Production build 0.71.5 2024