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
2 stars 2 forks source link

Fix unable to find config files #15

Closed shoshta73 closed 2 days ago

shoshta73 commented 2 days ago

Adds ability to use users shells native ability to autocomplete paths.

@gbowne1 are there any other improvements needed?

shoshta73 commented 2 days ago

This Implements checking for provided files via cli, and if there are none it defaults to interactive mode.

-c / --clean - checks if it should remove comments -s / --sort [a|d] - sorts, defaults to ascending mode

shoshta73 commented 2 days ago

I would also suggest to merge this with squash and merge option

gbowne1 commented 2 days ago

I can't think of any other improvments just yet. There's a bunch of features I would like to add but they would depend on a few things.

gbowne1 commented 2 days ago

Several of my systems don't have setuptools installed.. should that be included in requirements to make it easy for the user?

I just initially added stuff that would be helpful in development for formatting, linting, etc. in case the developer needed some tools they didnt have already.

shoshta73 commented 1 day ago

Setuptools package can be included in setup.py. I personally dont put them in setup.py files becase it is pretty easy to install them and post od the pyton packages that can be run as "standalone" application are usually installed in some local virtual enviroment contained somewhere in user facing filesystem (like $HOME/venv) or via pipx.

I can add them into the setup.py its quick change and if recall correctly it doesnt affect the structure of installation other than venv itself.

gbowne1 commented 1 day ago

no need if you don't think its necessary.. just was thinking of people with systems that didnt have this installed even though it would be simple to do the install themselves.