-
Notifications
You must be signed in to change notification settings - Fork 513
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix redis examples #5482
Fix redis examples #5482
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5482 +/- ##
==========================================
- Coverage 61.30% 61.28% -0.02%
==========================================
Files 312 312
Lines 11070 11072 +2
Branches 773 758 -15
==========================================
Hits 6786 6786
- Misses 4284 4286 +2 ☔ View full report in Codecov by Sentry. |
@@ -22,7 +22,34 @@ val elements: SCollection[(String, String)] = sc.redis(connectionOptions, keyPat | |||
|
|||
Looking up specific keys from redis can be done with @scaladoc[RedisDoFn](com.spotify.scio.redis.RedisDoFn): | |||
|
|||
@@snip [RedisExamples.scala](/scio-examples/src/main/scala/com/spotify/scio/examples/extra/RedisExamples.scala) { #RedisLookup_example } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we go away from integrated snippets in the future?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, but they can't be in the Examples files. I think this was the only one and I added it only last year when doing doc cleanup.
Adds missing
Example:
prefix needed to make redis example show up on the examples page, fixes usage formatting, moves documentation example into markdown to avoid it rendering as a header, removes scio-privateparDo
usage