Skip to content
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

Add parsing of all options in jfmt #769

Merged
merged 1 commit into from
Jul 16, 2023
Merged

Conversation

xexyl
Copy link
Contributor

@xexyl xexyl commented Jul 16, 2023

Updated usage message.

Added -m max_depth option. This was also added to the json_util_README.md document.

The option parsing for jfmt should be complete.

Updated version to 0.0.1.

The other tools will probably take more work to get the options parsed but this one was simple as the code already existed.

Updated usage message.

Added -m max_depth option. This was also added to the
json_util_README.md document.

The option parsing for jfmt should be complete.

Updated version to 0.0.1.

The other tools will probably take more work to get the options parsed
but this one was simple as the code already existed.
@lcn2 lcn2 merged commit 1725b03 into ioccc-src:master Jul 16, 2023
2 checks passed
@lcn2
Copy link
Contributor

lcn2 commented Jul 16, 2023

Thanks 🙏 @xexyl

Back to traveling in Canada 🇨🇦

@xexyl
Copy link
Contributor Author

xexyl commented Jul 16, 2023

Thanks 🙏 @xexyl

Happy to help.

Back to traveling in Canada 🇨🇦

Safe travels! And remember: the Road Goes Ever On!

I'm off to do other things but hopefully I can get another tool option parsing done soon. Soon most likely but whether that's tomorrow or not I don't know.

@xexyl xexyl deleted the jfmt-opts branch July 16, 2023 17:44
@xexyl
Copy link
Contributor Author

xexyl commented Jul 17, 2023

I am going to try resting now and then I hope to look at one of the other tools: whichever one seems like it will be quicker to implement the options.

Probably it will be jval because some of the options for that tool are also shared in jnamval.

But I don't know if I will finish the parsing or not: some new code will be required I think. Depends on how much time and energy I have!

Hope you're sleeping well or if you're awake that you're doing well! Well okay I hope you're doing well even if you're sleeping but you know what I mean!

Hopefully a commit in a few hours if not before that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants