findfirst should be consistent with Base.findfirst

sisl / Records.jl

Containers for object lists over time

Other

2 stars 2 forks source link

findfirst should be consistent with Base.findfirst #4

Closed MaximeBouton closed 5 years ago

MaximeBouton commented 5 years ago

The findfirst method implemented in this package returns 0 when it does not find the object. It should return nothing to be consistent with Base.findfirst.

rejuvyesh commented 5 years ago

Fixed with #5.