Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: include commas in the profile size, handle failure case of profile more than 500kB in size #5241

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
+23 −5

chore: add stat to check single profilesizes, restrict size to 500kb

349310f
Select commit
Loading
Failed to load commit list.
Open

fix: include commas in the profile size, handle failure case of profile more than 500kB in size #5241

chore: add stat to check single profilesizes, restrict size to 500kb
349310f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 6, 2024 in 1s

0.00% of diff hit (target 73.24%)

View this Pull Request on Codecov

0.00% of diff hit (target 73.24%)

Annotations

Check warning on line 84 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L84 

Added line #L84 was not covered by tests

Check warning on line 313 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L312-L313 

Added lines #L312 - L313 were not covered by tests

Check warning on line 323 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L323 

Added line #L323 was not covered by tests

Check warning on line 332 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L332 

Added line #L332 was not covered by tests

Check warning on line 348 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L341-L348 

Added lines #L341 - L348 were not covered by tests

Check warning on line 396 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L396 

Added line #L396 was not covered by tests

Check warning on line 403 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L403 

Added line #L403 was not covered by tests

Check warning on line 420 in router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

router/batchrouter/asyncdestinationmanager/klaviyobulkupload/klaviyobulkupload.go#L419-L420 

Added lines #L419 - L420 were not covered by tests
View more details on Codecov