Several comments have sentences with extra periods at the end

We have more comments with such a problem(look at the screenshot). I think the optimal way is fixing them all in this issue.
Set to "Needs work" for that.

Regexp: \w\.\.$

Thanks for the patch! If you set the issue status to "needs review" then the testbot will automatically test your patch for you.

I updated the title to more accurately reflect the wider scope of this issue.

Thank you for your work on cleaning up Drupal core's code style!

In order to fix core coding standards in a maintainable way, all our coding standards issues should be done on a per-rule basis across all of core, rather than fixing standards in individual modules or files. We should also separate fixes where we need to write new documentation from fixes where we need to correct existing standards. This all should be done as part of #2571965: [meta] Fix PHP coding standards in core, stage 1. A good place to start is the child issues of #2572645: [Meta] Fix 'Drupal.Commenting.FunctionComment' coding standard.

This one requires updates of the existing coder rules to detect multiple multiple fullstops. There are quite a few rules that detect fullstops on the end of comments. Once we've fixed the rules then we need to get the rules applied to core.

For background information on why we usually will not commit coding standards fixes that aren't scoped in that way, see the core issue scope guidelines, especially the note about coding standards cleanups. That document also includes numerous suggestions for scoping issues including documentation coding standards cleanups.

Contributing to the overall plan above will help ensure that your fixes for core's coding standards remain in core the long term.

I performed Novice Triage on this issue. I am leaving the Novice tag on this issue and added the Needs issue summary update tag because the issue summary should provide a little bit more detail based on @alexpott's comment regarding the next steps.

I think that this may be blocked on coder updates. Adding #2571965: [meta] Fix PHP coding standards in core, stage 1 as a parent issue might help and trying to find or create an issue to fix the rule in coder would also help relate the issues together.

1. A PR requested to coder added a sniffer, see https://github.com/pfrenssen/coder/pull/101
2. Found one more by using the sniffer

3. --- a/core/tests/Drupal/Tests/ComposerIntegrationTest.php
   +++ b/core/tests/Drupal/Tests/ComposerIntegrationTest.php
   @@ -79,7 +79,7 @@ public function testAllModulesReplaced() {
        // Get a list of all the files in the module path.
        $folders = scandir($module_path);
    
   -    // Make sure we only deal with directories that aren't . or ..
   +    // Make sure we only deal with directories that aren't '.' or '..'.
        $module_names = [];
        $discard = ['.', '..'];
        foreach ($folders as $file_name) {
   

   Used single quotes to bypass phpcbf correcting

Thanks for the updated patch.

1) The changes look good.

2) Patch applies cleanly.

[mac:kristen:drupal-8.9.x-dev]$ patch -p1 < 3117662-11.patch 
patching file composer/Generator/PackageGenerator.php
patching file core/lib/Drupal/Component/PhpStorage/FileStorage.php
patching file core/lib/Drupal/Core/Cache/Context/CacheContextsManager.php
patching file core/lib/Drupal/Core/Field/WidgetInterface.php
patching file core/lib/Drupal/Core/Form/FormBuilder.php
patching file core/lib/Drupal/Core/Routing/AccessAwareRouterInterface.php
patching file core/lib/Drupal/Core/Routing/StackedRouteMatchInterface.php
patching file core/modules/contact/tests/src/Functional/ContactSitewideTest.php
patching file core/modules/field/src/FieldStorageConfigInterface.php
patching file core/modules/locale/tests/src/Functional/LocaleImportFunctionalTest.php
patching file core/modules/migrate/src/Row.php
patching file core/modules/search/search.pages.inc
patching file core/modules/system/tests/src/Functional/Form/ElementTest.php
patching file core/modules/taxonomy/src/Plugin/views/argument_validator/Term.php
patching file core/modules/user/src/UserInterface.php
patching file core/modules/views/src/Plugin/views/display/PathPluginBase.php
patching file core/modules/views/src/Plugin/views/style/StylePluginBase.php
patching file core/modules/views/tests/src/Functional/Plugin/PagerTest.php
patching file core/modules/views_ui/tests/src/Functional/GroupByTest.php
patching file core/tests/Drupal/KernelTests/Core/Database/QueryTest.php
patching file core/tests/Drupal/Tests/ComposerIntegrationTest.php
patching file core/tests/Drupal/Tests/Core/Plugin/Discovery/TestDerivativeDiscoveryWithObject.php

3) I found one remaining line when searching the 8.9 dev codebase after patching so moving back to "Needs work".

core/misc/tabledrag.es6.js

    // Initialize the specified columns (for example, weight or parent columns)
    // to show or hide according to user preference. This aids accessibility
    // so that, e.g., screen reader users can choose to enter weight values and
    // manipulate form elements directly, rather than using drag-and-drop..
    self.initColumns();

Found a couple more:

• ./core/modules/simpletest/simpletest.module: @trigger_error(__FUNCTION__ . ' is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. Access the environment_cleaner service and call its cleanEnvironment() method, or use \Drupal\Core\Test\EnvironmentCleaner::cleanEnvironment() instead.. See https://www.drupal.org/node/3076634', E_USER_DEPRECATED);
• ./core/modules/simpletest/tests/src/Kernel/DeprecatedCleanupTest.php: * @expectedDeprecation simpletest_clean_environment is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. Access the environment_cleaner service and call its cleanEnvironment() method, or use \Drupal\Core\Test\EnvironmentCleaner::cleanEnvironment() instead.. See https://www.drupal.org/node/3076634

Hi Folks,

I have updated the patch to fix this coding in couple of the files below

Searching 15986 files for "\w\.\.$" (regex)

/var/www/html/drupalcorecopy/core/misc/tabledrag.es6.js:
  244      // to show or hide according to user preference. This aids accessibility
  245      // so that, e.g., screen reader users can choose to enter weight values and
  246:     // manipulate form elements directly, rather than using drag-and-drop..
  247      self.initColumns();
  248  

/var/www/html/drupalcorecopy/core/profiles/demo_umami/modules/demo_umami_content/default_content/images/home-grown-herbs.jpg:
    <binary>

/var/www/html/drupalcorecopy/core/tests/Drupal/Tests/Core/Asset/library_test_files/licenses.libraries.yml:
    1: # No license information: should default to GPL 2.0 or later..
    2  no-license-info:
    3    version: 1.0

/var/www/html/drupalcorecopy/core/themes/claro/js/tabledrag.es6.js:
  281      // to show or hide according to user preference. This aids accessibility
  282      // so that, e.g., screen reader users can choose to enter weight values and
  283:     // manipulate form elements directly, rather than using drag-and-drop..
  284      self.initColumns();
  285  

4 matches across 4 files

Thanks to @Kristen Pol for reviewing and thanks to @iyyappan.govind for submitting a new patch.

Todo: needs to address #16.

Based on @alexpott's comment in #9, let's postpone this waiting for #3126860: Add a sniff to fix duplicate full stops in comments landing first

Closed #3154903: Fix double periods at the end of sentences as a duplicate. Please credit @ju.vanderw here who filed it and submitted patch, and others if applicable. Thanks!

Removing Novice tag because this is postponed. Adding coding standards tag and changing to a task.

Closing as a duplicate of a later issue with an MR and moving credit