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

Exposing gas limit to chain config and using the parameter for create checkpoint in advance time #1498

Merged
merged 3 commits into from
May 29, 2024
Merged

Exposing gas limit to chain config and using the parameter for create checkpoint in advance time #1498

merged 3 commits into from
May 29, 2024

Conversation

slundqui
Copy link
Contributor

No description provided.

dpaiton
dpaiton approved these changes May 29, 2024
View reviewed changes
Copy link
Member

@dpaiton dpaiton left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I made an issue to remind us what we ultimately want (fewer parameters listed & in units of base) #1500 but this is fine for now to get stuff up & running.

@slundqui slundqui merged commit d0dd965 into delvtech:main May 29, 2024
3 checks passed
@slundqui slundqui deleted the expose_gas_limit branch May 29, 2024 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dpaiton dpaiton dpaiton approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@slundqui @dpaiton