Skip to content

fix(CHAIN-1870): add max block interval requirement#67

Merged
jackchuma merged 2 commits into
mainfrom
jack/max-block-interval
Aug 16, 2025
Merged

fix(CHAIN-1870): add max block interval requirement#67
jackchuma merged 2 commits into
mainfrom
jack/max-block-interval

Conversation

@jackchuma

Copy link
Copy Markdown
Collaborator

Fixes: For the Solana bridge, we can add a maximum value for block_interval_requirement when changing it so that the new block interval isn't too large. Maybe 1000 blocks?

@cb-heimdall

cb-heimdall commented Aug 16, 2025

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1
@jackchuma jackchuma requested review from mdehoog and xenoliss August 16, 2025 12:52
@jackchuma jackchuma merged commit f4ddeee into main Aug 16, 2025
3 checks passed
@jackchuma jackchuma deleted the jack/max-block-interval branch August 16, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants