You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jul 9, 2024. It is now read-only.
@bbenligiray at first I understood that we should not use tryMulticall on updateBeacons() when the count of update calldatas is equal to 1 but after re-reading the title and description on the issue I'm now thinking that the expected change is to just remove the tryMulticall altogether from updateBeacons(). Is this right?
If there are 11 Beacons to be updated and we're updating them in batches of 10, we can also omit the tryMulticall() of the second batch with a single Becaon update. It feels like this would also reduce code duplication in your implementation.
This adds an unnecessary gas cost overhead for individual Beacon updates, which is quite important for self-funded feeds
The text was updated successfully, but these errors were encountered: