Skip to content

refactor: optimize gas and modernize error handling in Signatures lib…#181

Closed
0xkuzeydurden wants to merge 1 commit intobase:mainfrom
0xkuzeydurden:patch-1
Closed

refactor: optimize gas and modernize error handling in Signatures lib…#181
0xkuzeydurden wants to merge 1 commit intobase:mainfrom
0xkuzeydurden:patch-1

Conversation

@0xkuzeydurden
Copy link

…rary

Replaced string based require messages with gas-efficient Custom Errors.

Implemented 'unchecked' blocks for loop increments to reduce gas consumption.

Optimized memory allocation in getApprovers by returning early when threshold is met.

…rary

Replaced string based require messages with gas-efficient Custom Errors.

Implemented 'unchecked' blocks for loop increments to reduce gas consumption.

Optimized memory allocation in getApprovers by returning early when threshold is met.
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jan 24, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/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 closed this Jan 30, 2026
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

Comments