ferrisetw icon indicating copy to clipboard operation
ferrisetw copied to clipboard

Schema::properties() isn't public

Open vvuk opened this issue 10 months ago • 1 comments

Schema::properties() is currently pub(crate), but is there any reason it shouldn't be just pub? It's very useful for being able to inspect the properties in a schema; I couldn't find any other way of getting this list.

vvuk avatar Apr 17 '24 15:04 vvuk

The properties were meant to be tied to the Parser so ideally you would use the Parser instance to access them. I'm not opposed to making this Schema::properties() if there're a good use case for this, or we could extend the Schema to cover this cases without directly exposing the properties. Let me know what you prefer @vvuk! Could you also give me some examples on use cases where you want to inspect the properties :)

n4r1b avatar Apr 18 '24 09:04 n4r1b