search

gbowne1 / json-maestro

JSONMaestro is a powerful tool designed for cleaning and processing JSON-like files. It simplifies tasks such as removing comments, eliminating duplicate keys, adding schema keys, and sorting keys. Ideal for developers working with configuration files and API responses, JSONMaestro enhances data integrity and prepares JSON data for further analysis
MIT License
3 stars 2 forks source link

[QAQC] test/test workflow / action fails #26

Closed gbowne1 closed 1 month ago

gbowne1 commented 1 month ago

The test/test workflow/action fails on PR run

shoshta73 commented 1 month ago

Im aware of this, trying to figure out how does json.load work, so we can have duplicate keys on load and not just last instance of the key

gbowne1 commented 1 month ago

I had to ponder the output of the run myself.

It's also saying that there is no package called jsonmaestro.

We should look at publishing at some point. There appears to be several methods..

Keep in mind there is both json.load and json.loads.

https://stackoverflow.com/questions/39719689/what-is-the-difference-between-json-load-and-json-loads-functions

shoshta73 commented 1 month ago

So technically speaking, json.loads should then fail to parse when given input with 2 identical key as its not in json spec as per rfc8259 (https://en.wikipedia.org/wiki/JSON#Interoperability)?

We should look at publishing at some point. There appears to be several methods..

Could you reserve the package name possibly?

gbowne1 commented 1 month ago

I'm just guessing that there could theoretically be a case where there are indeed identical keys, thought I haven't run across anything with that yet. However, I would opt for the case where the duplicated keys were not wanted. This could potentially be a feature we turn on and off with a option/flag.

Yes the package name could be reserved.