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.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not only ethereum net supporting (Gas Limit validation problem) #226

Open
d4mk0 opened this issue Mar 21, 2023 · 3 comments
Open

Not only ethereum net supporting (Gas Limit validation problem) #226

d4mk0 opened this issue Mar 21, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@d4mk0
Copy link
Contributor

d4mk0 commented Mar 21, 2023

Hi! 馃憢
Sometimes I use this lib to interact with different L2 networks. I.e. optimism, arbitrum

I found few issues, but now I see only one.
I dont know how we can except this problem, but it is problem, and 馃悞 (monkeypatching) only now can fix it.

Im about problem with tx gas limit validation.
Now https://github.com/q9f/eth.rb/blob/main/lib/eth/tx.rb#L206 we can only set 30kk limit, but sometimes in L2 we should set it higher.

I think it can be good first issue for some person.
Also we can discuss it in https://t.me/ruby_eth (about EVM like chains)

TY for u attention!

@q9f
Copy link
Owner

q9f commented Mar 22, 2023

Maybe a good thing would be to create a configuration for eth.rb that can be used and adjusted as we go.

Indeed, a good first issue :)

@q9f q9f added enhancement New feature or request good first issue Good for newcomers labels Mar 22, 2023
@MujahidAli954
Copy link

I want to work on this can you assign this to me

@q9f
Copy link
Owner

q9f commented Jul 13, 2023

Sure, let me know if you need anything!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants