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

					
				@ -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

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

Test / build (push) Successful in 9s

Required

Details

This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feature/181-q-json-flags:feature/181-q-json-flags

git checkout feature/181-q-json-flags

Sign in to join this conversation.

No reviewers

No milestone

No project

No assignees

2 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: RabbitLabs/random-bunny#227