Skip to content

N-02 Add LogBackfillBatchSize Validation#434

Open
dhaidashenko wants to merge 3 commits intodevelopfrom
fix/PLEX-2787-N-02-LogBackfillBatchSize-Validation
Open

N-02 Add LogBackfillBatchSize Validation#434
dhaidashenko wants to merge 3 commits intodevelopfrom
fix/PLEX-2787-N-02-LogBackfillBatchSize-Validation

Conversation

@dhaidashenko
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 17, 2026 16:18
@dhaidashenko dhaidashenko requested a review from a team as a code owner April 17, 2026 16:18
@github-actions
Copy link
Copy Markdown
Contributor

👋 dhaidashenko, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 17, 2026

📊 API Diff Results

No changes detected for module github.com/smartcontractkit/chainlink-evm

View full report

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds validation to prevent invalid LogBackfillBatchSize values in EVM chain TOML configuration.

Changes:

  • Validate LogBackfillBatchSize is greater than 0 during chain config validation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pkg/config/toml/config.go Outdated
@dhaidashenko dhaidashenko enabled auto-merge (squash) April 20, 2026 11:05
silaslenihan
silaslenihan previously approved these changes Apr 20, 2026
ilija42
ilija42 previously approved these changes Apr 20, 2026
@dhaidashenko dhaidashenko dismissed stale reviews from ilija42 and silaslenihan via 1e55866 April 24, 2026 12:27
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.

4 participants