Add explanatory comment

ocelotl · ocelotl · commit 58d12160be65 · 2023-09-07T20:24:14.000+02:00
diff --git a/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/_view_instrument_match.py b/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/_view_instrument_match.py
@@ -136,4 +136,8 @@ def collect(
                 )
                 if data_point is not None:
                     data_points.append(data_point)
+
+        # Returning here None instead of an empty list because the caller
+        # does not consume a sequence and to be consistent with the rest of
+        # collect methods that also return None.
         return data_points or None

Original file line number	Diff line number	Diff line change
`@@ -136,4 +136,8 @@ def collect(`
`136`	`136`	`)`
`137`	`137`	`if data_point is not None:`
`138`	`138`	`data_points.append(data_point)`
	`139`	`+`
	`140`	`+ # Returning here None instead of an empty list because the caller`
	`141`	`+ # does not consume a sequence and to be consistent with the rest of`
	`142`	`+ # collect methods that also return None.`
`139`	`143`	`return data_points or None`