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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(config): Cleanup & permit selective override #1940

Merged
merged 10 commits into from
Jan 2, 2025
Merged

Conversation

pxrl
Copy link
Contributor

@pxrl pxrl commented Dec 8, 2024

Permit certain config objects (i.e. MAX_BLOCK_LOOK_BACK) to be selectively overridden, rather than requiring a complete definition. This will slightly improve new chain automation.

Also simplify a few instances of config parsing.

Permit certain config objects (i.e. MAX_BLOCK_LOOK_BACK) to be selectively
overridden, rather than requiring a complete definition. This will slightly
improve new chain automation.

Also simplify a few instances of config parsing.
src/common/Config.ts Show resolved Hide resolved
src/common/Config.ts Outdated Show resolved Hide resolved
Copy link
Member

@nicholaspai nicholaspai left a comment

Choose a reason for hiding this comment

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

One refactoring suggestion!

src/common/Config.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@james-a-morris james-a-morris left a comment

Choose a reason for hiding this comment

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

one comment

@pxrl pxrl requested a review from nicholaspai December 18, 2024 14:26
@pxrl pxrl requested a review from dohaki as a code owner December 20, 2024 09:01
james-a-morris
james-a-morris previously approved these changes Jan 2, 2025
src/common/Config.ts Outdated Show resolved Hide resolved
nicholaspai
nicholaspai previously approved these changes Jan 2, 2025
@pxrl pxrl dismissed stale reviews from nicholaspai and james-a-morris via 75566e5 January 2, 2025 17:01
@pxrl pxrl merged commit 38e159d into master Jan 2, 2025
4 checks passed
@pxrl pxrl deleted the pxrl/cleanupConfig branch January 2, 2025 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants