v.info: add JSON support #3755
Merged
v.info: add JSON support #3755
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kritibirda26
commented
May 31, 2024
github-actions
bot
added
vector
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
You can go in the files tab of the PR, and choose multiple suggestions to add in the same commit. |
kritibirda26
force-pushed
the
add-json-to-v-info
branch
from
7ccf698 to
9ae3726
Compare
wenzeslaus
reviewed
Jun 4, 2024
|
Or follow instructions similar to #3763 (comment), I just walked through the same situation with another contributor a couple minutes ago |
kritibirda26
force-pushed
the
add-json-to-v-info
branch
from
c74840a to
4a252c4
Compare
wenzeslaus
requested changes
Jun 5, 2024
wenzeslaus
reviewed
Jun 5, 2024
wenzeslaus
reviewed
Jun 5, 2024
kritibirda26
force-pushed
the
add-json-to-v-info
branch
from
fbdd9a5 to
2b4d017
Compare
kritibirda26
commented
Jun 7, 2024
46 tasks
Use parson to add json output format support to the v.info module. The module has various flags to control the fields being output in case of plain format. The current prototype ignores the flags and always outputs all data for the JSON output.
Co-authored-by: Vaclav Petras <[email protected]>
kritibirda26
force-pushed
the
add-json-to-v-info
branch
from
132c4d3 to
159dbf6
Compare
cwhite911
suggested changes
Jun 17, 2024
|
Hi! I have updated the PR. |
wenzeslaus
reviewed
Jun 18, 2024
wenzeslaus
approved these changes
Jun 18, 2024
cwhite911
approved these changes
Jun 18, 2024
cyliang368
pushed a commit
to cyliang368/grass
that referenced
this pull request
Jun 22, 2024
Use parson to have JSON as an output format support. The module has various flags to control the fields being output in case of plain shell format controlled by flags. All (-get) of these are enabled when using format=shell. format=plain now allows use of flags. This may need special handling in the future to resolve the edge cases in a better way.
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Builds on #3744
Use parson to add json output format support to the v.info module. The module has various flags to control the fields being output in case of plain format. The current prototype ignores the flags and always outputs all data for the JSON output.