Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

perf: configure provider poll interval #8624

Merged

Conversation

mattsse
Copy link
Member

@mattsse mattsse commented Aug 7, 2024

we currently don't configure a chain specific poll interval and this would fallback to 7s if not local in alloy's default impl.

we can use the chain if it's known or assume some reasonable default

Copy link
Member

@DaniPopes DaniPopes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could upstream this logic to alloy

@mattsse mattsse merged commit 8390f2d into foundry-rs:master Aug 7, 2024
20 checks passed
benwjhack pushed a commit to CompassLabs/foundry-test that referenced this pull request Sep 11, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants