Skip to content

chore(sepolia): skip task origin validation for MAX_GAS_LIMIT upgrade#679

Merged
leopoldjoy merged 2 commits intomainfrom
sepolia/increase-max-gas-limit-signatures
May 7, 2026
Merged

chore(sepolia): skip task origin validation for MAX_GAS_LIMIT upgrade#679
leopoldjoy merged 2 commits intomainfrom
sepolia/increase-max-gas-limit-signatures

Conversation

@leopoldjoy
Copy link
Copy Markdown
Contributor

@leopoldjoy leopoldjoy commented May 7, 2026

Summary

  • Disable task origin verification for 2026-05-06-increase-max-gas-limit by setting skipTaskOriginValidation: true in both validation files
  • Remove signature files (creator, Base facilitator, SC facilitator) since they are no longer used
  • The git init + git apply patch approach creates a non-deterministic .git directory in lib/contracts/, causing tarball-based signature verification to fail across machines

Add creator, Base facilitator, and SC facilitator signatures for the
2026-05-06-increase-max-gas-limit task.
@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 7, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/2
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 2
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 2
2
1 if commit is unverified 0
Sum 2

Disable task origin verification (skipTaskOriginValidation: true) and
remove signature files. The git init + git apply patch approach creates
a non-deterministic .git directory in lib/contracts/, making tarball
signatures fail on other machines.
@leopoldjoy leopoldjoy changed the title chore(sepolia): add task origin signatures for MAX_GAS_LIMIT upgrade chore(sepolia): skip task origin validation for MAX_GAS_LIMIT upgrade May 7, 2026
@leopoldjoy leopoldjoy merged commit c8dc5e6 into main May 7, 2026
14 checks passed
@leopoldjoy leopoldjoy deleted the sepolia/increase-max-gas-limit-signatures branch May 7, 2026 23:37
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.

5 participants