add reflect for BinaryHeap

[cBournhonesque]

Objective

I wanted to have reflection for BinaryHeap for a personal project.

I'm running into some issues:

• I wanted to represent BinaryHeap as a reflect::List type since it's essentially a wrapper around a Vec, however there's no public way to access the underlying Vec, which makes it hard to implement the reflect::List methods. I have omitted the reflect::List methods for now.. I'm not sure if that's a blocker?
• what would be the alternatives? Simply not implement reflect::List? It is possible to implement FromReflect without it. Would the type be Struct then?

[alice-i-cecile]

Bit of a shame not to have the list-like behavior fully reflected, but it's fine to do this incrementally.

[james7132]

Please fix CI, there's a few errors and likely formatting that needs to be addressed.

[cBournhonesque]

I think it was erroring because I was still representing BinaryHeap as ReflectKind::List even though I could not implement reflect::List.

I switched it to just use ReflectKind::Value.

Although I'm not too sure how it works to add other bounds? I couldn't figure out how to add reflect(Debug, Serialize, Deserialize)

[MrGVSV]

Although I'm not too sure how it works to add other bounds? I couldn't figure out how to add reflect(Debug, Serialize, Deserialize)

Those can only be applied if you require that T also implements those traits. I don't think we'd want to be that restrictive, though.