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

Add ctx to vm calls, simplify Throttler logic. #2102

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Revert grpc updates

341151e
Select commit
Loading
Failed to load commit list.
Open

Add ctx to vm calls, simplify Throttler logic. #2102

Revert grpc updates
341151e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 18, 2024 in 0s

79.62% of diff hit (target 60.00%)

View this Pull Request on Codecov

79.62% of diff hit (target 60.00%)

Annotations

Check warning on line 30 in node/throttled_vm.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

node/throttled_vm.go#L30 

Added line #L30 was not covered by tests

Check warning on line 48 in node/throttled_vm.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

node/throttled_vm.go#L48 

Added line #L48 was not covered by tests

Check warning on line 97 in rpc/estimate_fee.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

rpc/estimate_fee.go#L96-L97 

Added lines #L96 - L97 were not covered by tests

Check warning on line 370 in rpc/trace.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

rpc/trace.go#L369-L370 

Added lines #L369 - L370 were not covered by tests

Check warning on line 50 in utils/throttler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

utils/throttler.go#L49-L50 

Added lines #L49 - L50 were not covered by tests

Check warning on line 63 in utils/throttler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

utils/throttler.go#L63 

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