ava-labs / ava-labs/Simplex

Rename Blacklist IsEmpty Method

Open
#375 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
22
Forks
4
Avg merge
2d 14h
Merged PRs (30d)
34

Description

True, but I actually think that was a mistake and we should've used `IsZero` for the blacklist as well.

I suggest we change it at a later time and just keep SimplexEpochInfo's IsZero as it's more more idiomatic and it's not a container (stack, list, slice) so it makes no sense to give it an "empty" adjective.

_Originally posted by @yacovm in https://github.com/ava-labs/Simplex/pull/363#discussion_r3182050358_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.