-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Supporting JSON file minification #14
Comments
Hello @pilotkid! I'm really sorry for this really late response. The action doesn't support the modification of JSON files. At the moment, this change will require some significant effort on my end to search for an appropriate library to use for the minification of the JSON files or if I'll need to create a small command to grep all whitespaces and necessary minifications for JSON. Accordingly, before implementing such a feature, I prefer to wait until more users request such a feature. If you're still in need of JSON minification, feel free please to open a pull request and ping me for review 😃 |
@nizarmah how exactly do we vote? I'll add a thumbs-up reaction |
Hey @stretch07. Yes, please, give a thumbs up to the original comment. If you have any suggestions (maybe a discussion post), let me know. |
Hello, I have a project that has a few JSON files in it. Does this action support the modification of JSON files?
The text was updated successfully, but these errors were encountered: