impl BorshSchema for HashSet

[tzemanovic]

closes #82

[frol]

As far as I remember, BorshSchema has never got to the usable state, and was considered experimental. Do you use it somewhere?

[tzemanovic]

As far as I remember, BorshSchema has never got to the usable state, and was considered experimental. Do you use it somewhere?

Oh, I didn't know, thanks! I'm currently trying to use it to generate encoding docs for https://github.com/anoma/anoma. Do you have bandwidth for reviewing PRs for it? I found some other limitations and I'd be happy to fix them

[frol]

@austinabell @miraclx Is there any decision regarding borsh schema or rather I should ask if you have any plans revisiting it when dealing with the interfaces on near-sdk-rs side?

[austinabell]

@austinabell @miraclx Is there any decision regarding borsh schema or rather I should ask if you have any plans revisiting it when dealing with the interfaces on near-sdk-rs side?

There are no guarantees that this format will be stable, but it's unclear at this time if or what the changes would need to be if any.

I don't have any issues with bringing this in that I can think of

[tzemanovic]

alright, thanks! I'd also like to add schema to BTreeSet, BTreeMap and PhantomData. Can I add them here or do you prefer separate PRs?

[frol]

I think it does not hurt to have this merged