search

gomoni / it

iterator library for go using a rangefunc experiment
MIT License
27 stars 2 forks source link

Mappable vs Mapable #16

Open wkhere opened 1 week ago

wkhere commented 1 week ago

Hi!

Your API has Mapable while it's spelled Mappable in your readme, so there seems to be confusion about the proper form. English is not my native language so I am not sure, Mappable looks more correct. Plus this: https://en.wiktionary.org/wiki/mapability "misspelling of mappability".

Do you think it's ok to have yet another API change?

vyskocilm commented 1 week ago

Hi,

it's not mine mother tongue as well :-). It looks like the variant with double p is the correct: https://en.wiktionary.org/wiki/mappable

As there are no other packages using it according pkg.go.dev and no v1 has been released, feel free to sent a PR.

Thanks for looking at this.