search

metafacture / metafacture-core

Core package of the Metafacture tool suite for metadata processing.
https://metafacture.org
Apache License 2.0
69 stars 34 forks source link

Implement JSON booleans and numbers. #458

Closed blackwinter closed 1 year ago

blackwinter commented 2 years ago

Resolves #457.

blackwinter commented 2 years ago

One minor suggestion, maybe for the tests ? would be a good boolean marker?

Actually, I don't think it would. Since ? is a path wildcard, it would probably lead to confusion. Otherwise, I would've used it already ;) (We have * as an alternative array marker in the tests which, in hindsight, probably wasn't such a good idea either.)

fsteeg commented 2 years ago

Since ? is a path wildcard, it would probably lead to confusion.

Oh, you're right, I forgot about that.

TobiasNx commented 1 year ago

+1 works nicely :)