Update JS library domready to version 1.0.8

When updating include the minified file, the original source and the source map file (if any supplied by the lib).

Updated to new version, and source file added.

We need to update the tests too at \Drupal\system\Tests\Common\AttachedAssetsTest::testVersionQueryString

function testVersionQueryString() {
    $build['#attached']['library'][] = 'core/backbone';
    $build['#attached']['library'][] = 'core/domready';
    $assets = AttachedAssets::createFromRenderArray($build);

    $js = $this->assetResolver->getJsAssets($assets, FALSE)[1];
    $js_render_array = \Drupal::service('asset.js.collection_renderer')->render($js);
    $rendered_js = $this->renderer->render($js_render_array);
-    $this->assertTrue(strpos($rendered_js, 'core/assets/vendor/backbone/backbone-min.js?v=1.1.2') > 0 && strpos($rendered_js, 'core/assets/vendor/domready/ready.min.js?v=1.0.7') > 0 , 'JavaScript version identifiers correctly appended to URLs');
+   $this->assertTrue(strpos($rendered_js, 'core/assets/vendor/backbone/backbone-min.js?v=1.1.2') > 0 && strpos($rendered_js, 'core/assets/vendor/domready/ready.min.js?v=1.0.8') > 0 , 'JavaScript version identifiers correctly appended to URLs');
  }

EDIT: I figured out that it was actually changed, investigating now why failing.

We need to update the core.libraries.yml file. I haven't worked with external JS libraries before, so I'm not sure do we need to provide the source file by executing something?

diff --git a/core/core.libraries.yml b/core/core.libraries.yml
index dd8eae7..cf0f37a 100644
--- a/core/core.libraries.yml
+++ b/core/core.libraries.yml
@@ -34,10 +34,10 @@ ckeditor:
 
 domready:
   remote: https://github.com/ded/domready
-  version: "1.0.7"
+  version: "1.0.8"
   license:
     name: MIT
-    url: https://github.com/ded/domready/blob/v1.0.7/LICENSE
+    url: https://github.com/ded/domready/blob/v1.0.8/LICENSE
     gpl-compatible: true
   js:
     assets/vendor/domready/ready.min.js: { weight: -21, minified: true }

Tested, everything is ok

how i tested it manually:

• fund that statistics module is using domready
• tested if statistics is saving page hits with patch
• without it.

--- /dev/null
+++ b/core/assets/vendor/domready/ready.js

--- a/core/assets/vendor/domready/ready.min.js
+++ b/core/assets/vendor/domready/ready.min.js

@nod_ given that the minified is not minified should we be bothering to in the unminified file.

Umm yeah, let's loose the unminified file for that one. I don't want to add the unminified version of modernizr or ckeditor or any other lib we don't have a map file for.

Thanks, all good.

Nice reduction in size with their new minification config.

Nice reduction in size with their new minification config.

Wow, yes, significantly smaller!

Committed 9489b81 and pushed to 8.0.x. Thanks!