Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

chore: reduce the default EVM gas limit #497

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Sep 26, 2023

Overview

Currently, relaying attestations takes around 550000 gas, and deploying the contracts around 1250000 gas. So, we can reduce the default gas limit to a more realistic value.

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id added the evm label Sep 26, 2023
@rach-id rach-id requested a review from adlerjohn September 26, 2023 19:05
@rach-id rach-id self-assigned this Sep 26, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #497 (e8aea19) into main (73b0565) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #497   +/-   ##
=======================================
  Coverage   25.37%   25.37%           
=======================================
  Files          29       29           
  Lines        2834     2834           
=======================================
  Hits          719      719           
  Misses       2029     2029           
  Partials       86       86           
Files Coverage Δ
evm/evm_client.go 52.25% <ø> (ø)

@rach-id rach-id merged commit f4ce117 into celestiaorg:main Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants