lib: Add a standard parser option for JSON formatting

[kritibirda26]

As a part of #3019, JSON format support will be added to multiple modules. By default, modules output in existing plain format. If the format=json option is provided, modules will output in JSON format instead. To avoid duplication of code across several modules, define a standard parser option.

[petrasovaa]

Please also replace it in v.db.select, the options are different there, so you keep the current v.db.select options and descriptions.

I would think the format should be by default required, although since it has default answer, it probably doesn't matter that much.

[wenzeslaus]

This is great improvement @kritibirda26. To fix the OSGeo4W CI, please update your branch from main (e.g. Update branch on GitHub or merge main locally into this branch).

[kritibirda26]

@petrasovaa Hi! I have updated v.db.select and also changed the option to be required by default.

[kritibirda26]

The tests are failing with this weird error but the build passes locally. Also, the string that is said to be invalid is actually a valid isoformat string. Any suggestions on how to debug?

gnu/docs/html/g.gui.timeline.html
Traceback (most recent call last):
  File "/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/utils/mkhtml.py", line 921, in <module>
    git_commit = get_last_git_commit(
  File "/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/utils/mkhtml.py", line 397, in get_last_git_commit
    return parse_git_commit(
  File "/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/utils/mkhtml.py", line 242, in parse_git_commit
    git_log["date"] = format_git_commit_date_from_local_git(
  File "/home/runner/work/grass/grass/dist.x86_64-pc-linux-gnu/utils/mkhtml.py", line 345, in format_git_commit_date_from_local_git
    return datetime.fromisoformat(
ValueError: Invalid isoformat string: '2024-05-17T16:09:45Z'

https://github.com/OSGeo/grass/actions/runs/9131120968/job/25109628154?pr=3704#step:8:6469

[neteler]

A guess: it probably expects a string without the 'Z' character?

[kritibirda26]

@neteler I see, do you know if there is a local git setting I can configure to fix the datetime format for the commit timestamp?

[wenzeslaus]

This is failing across several PRs with repeated runs. I'm opening a separate issue to discuss this.

[echoix]

What prefix for the title of the PR should be used before merging, once it is rebased? (the build issue is most probably fixed now on main).

[kritibirda26]

Thanks all for reviewing the PR and helping fix the CI issues. Please let me know me if any other changes are needed in this PR.

[echoix]

@petrasovaa previously requested changes, she might want to place an approval to overrule her review.

And a prefix for the PR, so it can be classified in the release notes. I'm not sure which one to apply.