Skip to content

Resolve credo warnings - #1

Merged
vbaranov merged 1 commit into
mainfrom
vb-resolve-credo-warnings
Oct 19, 2026
Merged

vbaranov merged 1 commit into
mainfrom
vb-resolve-credo-warnings

Conversation

@vbaranov

@vbaranov vbaranov commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Silence credo warnings:

  Warnings - please take a look                                                                                                                                           
┃ 
┃ [W] ↗ Using `length/1` is expensive, prefer comparing against an empty list.
┃       lib/ex_eth_bls.ex:192:14 #(ExEthBls.aggregate_verify)
┃ [W] ↗ Using `length/1` is expensive, prefer comparing against an empty list.
┃       lib/ex_eth_bls.ex:173:37 #(ExEthBls.fast_aggregate_verify)
┃ [W] ↗ Using `length/1` is expensive, prefer comparing against an empty list.
┃       lib/ex_eth_bls.ex:147:69 #(ExEthBls.aggregate_signatures)
┃ [W] ↗ Using `length/1` is expensive, prefer comparing against an empty list.
┃       lib/ex_eth_bls.ex:122:72 #(ExEthBls.aggregate_public_keys)

@vbaranov
vbaranov force-pushed the vb-resolve-credo-warnings branch from f92d80c to 4c872a4 Compare October 19, 2026 12:47
@vbaranov
vbaranov force-pushed the vb-resolve-credo-warnings branch from 4c872a4 to 9e3be74 Compare October 19, 2026 12:54
@vbaranov
vbaranov merged commit 0d71250 into main Oct 19, 2026
2 checks passed
@vbaranov
vbaranov deleted the vb-resolve-credo-warnings branch October 19, 2026 13:00
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.

1 participant