RabbitLabs/random-bunny
5
0
Fork 0
Code Issues 9 Pull requests 11 Projects 1 Releases 18 Wiki Activity Actions

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

Merged
Vylpes merged 17 commits from feature/181-q-json-flags into develop 2024-12-07 22:21:43 +00:00
Conversation 1 Commits 17 Files changed 3 +1 -3
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 3e4c3d502a - Show all commits

4
src/helpers/outputHelper.ts
View file

@ -20,8 +20,7 @@ export default class OutputHelper {
return this.GetFriendlyObjectText(outputObject);
}
/* eslint-disable @typescript-eslint/no-explicit-any */
private static GetFriendlyObjectText(object: any): string {
private static GetFriendlyObjectText<T>(object: T): string {
const output: string[] = [];
Vylpes marked this conversation as resolved Outdated
Copilot commented 2024-11-18 20:51:36 +00:00
Outdated
Review
Copy link

[nitpick] The use of 'any' for the object parameter is not ideal. Consider using a more specific type.

[nitpick] The use of 'any' for the object parameter is not ideal. Consider using a more specific type.
Vylpes marked this conversation as resolved Outdated
VylpesTester commented 2024-10-26 21:58:16 +01:00
Outdated
Review
Copy link

I don't think I like the way this is done, I think it should be how it was before, but when its actually printed to the terminal it should remove the appended new line

I don't think I like the way this is done, I think it should be how it was before, but when its actually printed to the terminal it should remove the appended new line
Vylpes commented 2024-11-27 18:34:02 +00:00
Outdated
Review
Copy link

Not exactly sure what this means? It appears to be working fine and looks like that to me

Not exactly sure what this means? It appears to be working fine and looks like that to me
for (const key in object) {
@ -30,5 +29,4 @@ export default class OutputHelper {
return output.join("\n");
}
/* eslint-enable @typescript-eslint/no-explicit-any */
}