RabbitLabs/random-bunny
4
0
Fork 0
Code Issues 10 Pull requests 8 Actions Projects 1 Releases 18 Wiki Activity

Fix bug where you weren't able to use both the -q and -j flags at the same time #227

Open
Vylpes wants to merge 3 commits from feature/181-q-json-flags into develop
pull from: feature/181-q-json-flags
merge into: RabbitLabs:develop
Conversation 0 Commits 3 Files changed 3 +9
1 changed files with 9 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit af30ab5ea7 - Show all commits

9
src/helpers/outputHelper.ts
View file

@ -8,6 +8,15 @@ export default class OutputHelper {
const outputLines: string[] = [];
if (options.json) {
if (options.queryMetadata != null) {
VylpesTester commented 2024-09-15 16:19:09 +01:00
Review
Copy link

I feel like there could be a better way to do this? As now we have the output kinda splintered. I suggest maybe we have the json object created in a separate function, and then split to human-readable if -j isn't present

I feel like there could be a better way to do this? As now we have the output kinda splintered. I suggest maybe we have the json object created in a separate function, and then split to human-readable if `-j` isn't present
return JSON.stringify({
...result,
query: {
...response.Query,
}
})
}
return JSON.stringify(result);
}