-
Notifications
You must be signed in to change notification settings - Fork 66
Move 'sink' module to this codebase, add webhook sink #658
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
Conversation
… imported (same license already top level)
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.
Small nitpick
sink/webhook/cli_test.go
Outdated
@@ -0,0 +1,120 @@ | |||
package webhook |
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.
What is this file?
cmd/substreams/gui.go
Outdated
sink.AddFlagsToSet(guiCmd.Flags(), | ||
sink.FlagIgnore(sink.FlagDevelopmentMode, | ||
sink.FlagLiveBlockTimeDelta, | ||
sink.FlagInfiniteRetry)) | ||
|
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.
Formatting issue?
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.
I think it's just because we would need to:
- Add a new line before
sink.FlagDevelopmentMode
- Add a trailing comma and newline after
sink.FlagInfiniteRetry
That would make the rendering better. There is two other occurrences of this.
sink
module from https://github.com/streamingfast/substreams-sinksubstreams run
command as an implementation of that sinkwebhook
sinkgithub.com/streamingfast/substreams/sink
instead ofgithub.com/streamingfast/substreams-sink
with minimal changes.