Skip to content

chore: review comments

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

84.97% of diff hit (target 74.82%)

View this Pull Request on Codecov

84.97% of diff hit (target 74.82%)

Annotations

Check warning on line 281 in warehouse/integrations/azure-synapse/azure-synapse.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/integrations/azure-synapse/azure-synapse.go#L281 

Added line #L281 was not covered by tests

Check warning on line 612 in warehouse/integrations/azure-synapse/azure-synapse.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/integrations/azure-synapse/azure-synapse.go#L612 

Added line #L612 was not covered by tests

Check warning on line 954 in warehouse/integrations/azure-synapse/azure-synapse.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/integrations/azure-synapse/azure-synapse.go#L953-L954 

Added lines #L953 - L954 were not covered by tests

Check warning on line 632 in warehouse/integrations/mssql/mssql.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/integrations/mssql/mssql.go#L632 

Added line #L632 was not covered by tests

Check warning on line 969 in warehouse/integrations/mssql/mssql.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/integrations/mssql/mssql.go#L968-L969 

Added lines #L968 - L969 were not covered by tests

Check warning on line 1303 in warehouse/integrations/redshift/redshift.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/integrations/redshift/redshift.go#L1302-L1303 

Added lines #L1302 - L1303 were not covered by tests

Check warning on line 445 in warehouse/router/identities.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/router/identities.go#L442-L445 

Added lines #L442 - L445 were not covered by tests

Check warning on line 447 in warehouse/router/identities.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/router/identities.go#L447 

Added line #L447 was not covered by tests

Check warning on line 34 in warehouse/router/sync.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/router/sync.go#L33-L34 

Added lines #L33 - L34 were not covered by tests

Check warning on line 39 in warehouse/router/sync.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/router/sync.go#L38-L39 

Added lines #L38 - L39 were not covered by tests

Check warning on line 297 in warehouse/router/upload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/router/upload.go#L297 

Added line #L297 was not covered by tests

Check warning on line 284 in warehouse/source/source.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/source/source.go#L284 

Added line #L284 was not covered by tests

Check warning on line 292 in warehouse/source/source.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/source/source.go#L292 

Added line #L292 was not covered by tests

Check warning on line 31 in warehouse/utils/uploader.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/utils/uploader.go#L31 

Added line #L31 was not covered by tests

Check warning on line 36 in warehouse/utils/uploader.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/utils/uploader.go#L36 

Added line #L36 was not covered by tests

Check warning on line 576 in warehouse/validations/validate.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

warehouse/validations/validate.go#L576 

Added line #L576 was not covered by tests
View more details on Codecov