Skip to content

chore: review comments

Codecov / codecov/patch succeeded Jan 8, 2025 in 0s

86.41% of diff hit (target 74.74%)

View this Pull Request on Codecov

86.41% of diff hit (target 74.74%)

Annotations

Check warning on line 75 in router/batchrouter/asyncdestinationmanager/snowpipestreaming/channel.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/snowpipestreaming/channel.go#L75

Added line #L75 was not covered by tests

Check warning on line 169 in router/batchrouter/asyncdestinationmanager/snowpipestreaming/channel.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/snowpipestreaming/channel.go#L169

Added line #L169 was not covered by tests

Check warning on line 194 in router/batchrouter/asyncdestinationmanager/snowpipestreaming/channel.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/snowpipestreaming/channel.go#L194

Added line #L194 was not covered by tests

Check warning on line 113 in router/batchrouter/asyncdestinationmanager/snowpipestreaming/snowpipestreaming.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/snowpipestreaming/snowpipestreaming.go#L112-L113

Added lines #L112 - L113 were not covered by tests

Check warning on line 117 in router/batchrouter/asyncdestinationmanager/snowpipestreaming/snowpipestreaming.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/snowpipestreaming/snowpipestreaming.go#L116-L117

Added lines #L116 - L117 were not covered by tests

Check warning on line 181 in router/batchrouter/asyncdestinationmanager/snowpipestreaming/snowpipestreaming.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/snowpipestreaming/snowpipestreaming.go#L180-L181

Added lines #L180 - L181 were not covered by tests