lostluck commented on a change in pull request #11690:
URL: https://github.com/apache/beam/pull/11690#discussion_r424681041
##########
File path: learning/katas/go/Core Transforms/Map/ParDo Struct/task.md
##########
@@ -18,16 +18,16 @@
-->
# ParDo - using a struct as a DoFn
-In previous katas, we provided our ParDo a `func` as a DoFn. In this example,
we will explore
-the use of a `struct` to in our transform.
+In previous katas, we provided `beam.ParDo` a `func` as a DoFn. In this
example, we will explore
Review comment:
I guess I could go either way?
Including the identifier "beam." is more specific and I like it better
visually. (especially with the `codefont` indication).
Reading it aloud it is certainly more of a mouthful than without "beam.",
but that's should only be a problem for very novice readers who aren't
subtising the words they read yet.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
