+++ b/core/includes/config.inc
@@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf
+ * for all objects of this type. If it doesn't contain it, we load first the generic metadata
Minor: We shouldn't abbreviate here, i.e. "doesn't" -> "does not"
+++ b/core/includes/config.inc
@@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf
+ * 1. Load metadata for image.style.*
+ * 2. Merge metadata for image.style.large
I think both of these should be "from" not "for". In general, I think it would be clearer to put the listing in words, rather than using this actual list. That's definitely debatable, though.
+++ b/core/includes/config.inc
@@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf
+function config_meta_data($name, $key = '') {
This is missing some @return documentation.
+++ b/core/includes/config.inc
@@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf
+ else {
+ return $meta[$name];
+ }
Might as well drop the whole else, since the if already returns.
+++ b/core/includes/config.inc
@@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf
+function config_meta_build($name, $data) {
This is missing docs on @params and @return.
+++ b/core/includes/config.inc
@@ -105,6 +106,142 @@ function config_sync_get_changes(StorageInterface $source_storage, StorageInterf
+function _config_meta_element($meta, $data, $base, $index, $group = '') {
Is missing @return documentation.
+++ b/core/modules/system/meta/system.site.yml
@@ -0,0 +1,18 @@
\ No newline at end of file
So let's add one :-)