Remove Book module from comments not in tests

Applied the MR and ran the search and there are 4 returns

1 core/lib/Drupal/Core/Database/Connection.php = Not related to module
2 core/lib/Drupal/Core/Password/PhpPassword.php = Not related to module
3 core/modules/comment/comment.module = Not related to module
4 core/modules/comment/tests/src/Functional/CommentBookTest.php = IS related to the module

I'm making an assumption that CommentBookTest will be fixed when tests are moved.

MR no longer applies and needs rebasing.

Rebase seems good.

test.sh from the hook_help accessibility issue has been accidentally added.

My mistake in #8, saw green.

See test.sh is removed.

This looks good overall. I left one suggestion on the MR for a comment that went weird, and another question.

Thanks!

Also, I looked at the remaining usages and I think we should also update this bit from comment.module:

* Users can post comments to discuss a story, collaborative book page, user    
 * etc.  

Also, this issue seems to address docblock comments, but there are still a number of inline comments that also reference the book module:

[ayrton:drupal | Sun 10:18:18] $ grep -ri "\bbook\b" * | grep "//" |  grep -v "core/modules/book" | grep -v "vendor" | grep -v "node_modules" | grep -vi "migrate"
core/tests/Drupal/KernelTests/Core/Asset/LibraryDiscoveryIntegrationTest.php:    // Simulate starterkit_theme defining the book-navigation library.
core/tests/Drupal/KernelTests/Core/Asset/LibraryDiscoveryIntegrationTest.php:    // Activate the theme that extends the book-navigation library in
core/lib/Drupal/Core/Database/Connection.php: * @see http://php.net/manual/book.pdo.php
core/lib/Drupal/Core/Password/PhpPassword.php: * @see https://www.php.net/manual/en/book.password.php
core/modules/image/tests/src/Functional/ImageStylesPathAndUrlTest.php:        // (cf. http://www.libpng.org/pub/png/book/chapter08.html#png.ch08.div.2)
core/modules/system/tests/src/Functional/UpdateSystem/UpdatePathTestBaseFilledTest.php:    // Make sure the book node exists.
core/modules/system/tests/src/Functional/UpdateSystem/UpdatePathTestBaseFilledTest.php:    // Ensure that the Book module's node type does not have duplicated enforced
core/modules/system/src/Plugin/ImageToolkit/GDToolkit.php:      $descriptions[] = $this->t('The GD Library for PHP is enabled, but was compiled without support for functions used by the rotate and desaturate effects. It was probably compiled using the official GD libraries from the <a href="https://libgd.github.io/">gdLibrary site</a> instead of the GD library bundled with PHP. You should recompile PHP --with-gd using the bundled GD library. See <a href="https://www.php.net/manual/book.image.php">the PHP manual</a>.');
core/modules/views/tests/src/Kernel/TestViewsTest.php:    // `node.type.book` config entity is a config dependency.
core/modules/views/tests/src/Kernel/TestViewsTest.php:    // `node.type.book` config entity is a config dependency.

Some of those are not relevant, but others definitely reference the Book module.

Saving credits.

Changes appear to have been addressed.

Liked the replacement of "my_module.settings" that should stand the test of time and any future deprecations.

Thanks folks.

Looks good to me

Committed to 11.x