web-platform-dx · foolip · Jul 5, 2024 · ddbeck · Jul 8, 2024 · foolip
diff --git a/features/avif.yml.dist b/features/avif.yml.dist
@@ -2,3 +2,5 @@
 # Do not edit this file by hand. Edit the source file instead!
 
 {}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/http2.yml.dist b/features/http2.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: http2.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/http3.yml.dist b/features/http3.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: http3.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/intersection-observer-v2.yml.dist b/features/intersection-observer-v2.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: intersection-observer-v2.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/jpegxl.yml.dist b/features/jpegxl.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: jpegxl.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/mathml.yml.dist b/features/mathml.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: mathml.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/notifications.yml.dist b/features/notifications.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: notifications.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/scroll-to-text-fragment.yml.dist b/features/scroll-to-text-fragment.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: scroll-to-text-fragment.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/webdriver-bidi.yml.dist b/features/webdriver-bidi.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: webdriver-bidi.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/features/webp.yml.dist b/features/webp.yml.dist
@@ -0,0 +1,6 @@
+# Generated from: webp.yml
+# Do not edit this file by hand. Edit the source file instead!
+
+{}
+
+# Nothing is generated, nothing to see here.
diff --git a/scripts/dist.ts b/scripts/dist.ts
@@ -216,6 +216,10 @@ function toDist(sourcePath: string): YAML.Document {
     insertCompatFeatures(dist, sortedGroups);
   }
 
+  if (dist.contents.items.length === 0) {
+    dist.comment = ' Nothing is generated, nothing to see here.'
+  }
+
   return dist;
 }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -2,3 +2,5 @@
		# Do not edit this file by hand. Edit the source file instead!

		{}

		# Nothing is generated, nothing to see here.
Copy link Collaborator ddbeck Jul 8, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. What do you think of linking out to some documentation (you can just stub a heading for now, if you like) to handle the whole "why is it empty?" question? Copy link Collaborator Author foolip Jul 9, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. A link to https://github.com/web-platform-dx/web-features/blob/5f17741fd7c73119a8e25270ca2070d04d73cc0c/docs/CONTRIBUTING.md#feature-dist-files after it's landed, perhaps? Copy link Collaborator Author foolip Jul 9, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Although, maybe it's better to link to the docs at the top of all dist files? Copy link Contributor captainbrosset Jul 10, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. I agree with this. Linking to the docs on line 3 of all dist files (right after asking people not to edit the file by hand) would give them a friendly way to learn more, especially for folks who prefer to jump in the code first, and read docs later. Copy link Collaborator Author foolip Jul 10, 2024 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Cool, I've sent #1345 as a draft, to land after #1330. If the docs mention `{}` then I don't think this PR is needed at all.