Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow user to specify a format for stdout #242

Closed
nehanene15 opened this issue Apr 28, 2021 · 1 comment
Closed

Allow user to specify a format for stdout #242

nehanene15 opened this issue Apr 28, 2021 · 1 comment
Assignees
@yogeshtewari
Labels
priority: p1 High priority. Fix may be included in the next release. Release Next release candidate type: feature request 'Nice-to-have' improvement, new feature or different behavior or design.

Comments

@nehanene15
Copy link
Collaborator

Currently, the default stdout output is not readable due to the large amount of fields we print. The user should be able to specify a --format flag similar to the gcloud format flag.

We can also trim down the amount of fields we output to stdout if they are user specified parameters.
@nehanene15 nehanene15 added the priority: p1 High priority. Fix may be included in the next release. label Apr 28, 2021
yogeshtewari pushed a commit that referenced this issue Jun 29, 2021
feat: Allow user to specify a format for stdout (#242)
77ba13d
@yogeshtewari
Copy link
Contributor

Updated code to take in --format flag and pretty print formatted results on the console. Default format is tabular grid.

@yogeshtewari yogeshtewari linked a pull request Jun 29, 2021 that will close this issue
feat: Allow user to specify a format for stdout (#242) #275
Closed
@freedomofnet freedomofnet added the type: feature request 'Nice-to-have' improvement, new feature or different behavior or design. label Jul 14, 2021
@nehanene15 nehanene15 added the Release Next release candidate label Jul 15, 2021
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
feat: Allow user to specify a format for stdout (#242)
28f983f
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
feat: Allow user to specify a format for stdout, tabulate dependency (#…
cc384f1 
…242)
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
feat: Allow user to specify a format for stdout, added for other test…
76fbea5 
… cases (#242)
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
Revert "feat: Allow user to specify a format for stdout, added for ot…
dc3d094 
…her test cases (#242)"

This reverts commit 76fbea5.

adding default parameter value for format
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
feat: Allow user to specify a format for stdout, added default parame…
f41759b 
…ter for format (#242)
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
feat: Allow user to specify a format for stdout, format for text resu…
1259529 
…lt handler (#242)
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
feat: Allow user to specify a format for stdout, linting fix (#242)
5707b3d
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
feat: Allow user to specify a format for stdout, files reformatted wi…
f730ac9 
…th black (#242)
yogeshtewari pushed a commit that referenced this issue Jul 30, 2021
Revert "feat: Allow user to specify a format for stdout, files reform…
bb5f91e 
…atted with black (#242)"

This reverts commit f730ac9.
yogeshtewari pushed a commit that referenced this issue Aug 2, 2021
feat: Allow user to specify a format for stdout, files reformatted wi…
4522073 
…th black version 19 (#242)
yogeshtewari pushed a commit that referenced this issue Aug 2, 2021
feat: Allow user to specify a format for stdout, updated readme and f…
bb6610d 
…ormat short form (#242)
yogeshtewari pushed a commit that referenced this issue Aug 2, 2021
feat: Allow user to specify a format for stdout, updated help for for…
3e64e34 
…mat (#242)
yogeshtewari pushed a commit that referenced this issue Aug 2, 2021
feat: Allow user to specify a format for stdout, default format is ta…
d5ff705 
…ble (#242)
yogeshtewari added a commit that referenced this issue Aug 3, 2021
@yogeshtewari @nehanene15
feat: Allow user to specify a format for stdout (#242) (#293)
f0a9fa1 
* feat: Allow user to specify a format for stdout (#242)

* feat: Allow user to specify a format for stdout, tabulate dependency (#242)

* feat: Allow user to specify a format for stdout, added for other test cases (#242)

* Revert "feat: Allow user to specify a format for stdout, added for other test cases (#242)"

This reverts commit 76fbea5.

adding default parameter value for format

* feat: Allow user to specify a format for stdout, added default parameter for format (#242)

* feat: Allow user to specify a format for stdout, format for text result handler (#242)

* feat: Allow user to specify a format for stdout, linting fix (#242)

* feat: Allow user to specify a format for stdout, files reformatted with black (#242)

* Revert "feat: Allow user to specify a format for stdout, files reformatted with black (#242)"

This reverts commit f730ac9.

* feat: Allow user to specify a format for stdout, files reformatted with black version 19 (#242)

* feat: Allow user to specify a format for stdout, updated readme and format short form (#242)

* feat: Allow user to specify a format for stdout, updated help for format (#242)

* Update README.md

Co-authored-by: Neha Nene <[email protected]>

* feat: Allow user to specify a format for stdout, default format is table (#242)

Co-authored-by: Yogesh Tewari <[email protected]>
Co-authored-by: Neha Nene <[email protected]>
nehanene15 added a commit that referenced this issue Aug 4, 2021
@nehanene15
Revert "feat: Allow user to specify a format for stdout (#242) (#293)"
1bbd6b6 
This reverts commit f0a9fa1.
yogeshtewari added a commit that referenced this issue Aug 4, 2021
@yogeshtewari @nehanene15
Revert "feat: Allow user to specify a format for stdout (#242) (#293)" (
5fe6a8d 
#295)

This reverts commit f0a9fa1.

Co-authored-by: Neha Nene <[email protected]>
yogeshtewari pushed a commit that referenced this issue Aug 5, 2021
feat: Allow user to specify a format for stdout(#242)
fb76f44
yogeshtewari pushed a commit that referenced this issue Aug 5, 2021
feat: Allow user to specify a format for stdout. Test case fixes(#242)
e37a88a
yogeshtewari pushed a commit that referenced this issue Aug 5, 2021
feat: Allow user to specify a format for stdout. Test case fixes(#242)
48b1581
yogeshtewari pushed a commit that referenced this issue Aug 10, 2021
feat: Allow user to specify a format for stdout. Default format and T…
51f14e0 
…ext result handler test case fixes(#242)
yogeshtewari added a commit that referenced this issue Aug 25, 2021
@yogeshtewari
feat: Allow user to specify a format for stdout T2 (#242) (#296)
ec1af22 
* feat: Allow user to specify a format for stdout(#242)

* feat: Allow user to specify a format for stdout. Test case fixes(#242)

* feat: Allow user to specify a format for stdout. Test case fixes(#242)

* feat: Allow user to specify a format for stdout. Default format and Text result handler test case fixes(#242)

* feat: Trim the number of columns we output to stdout (#294)

* feat: Trim the number of columns we output to stdout (#294)

* feat: Trim the number of columns we output to stdout (#294)

Co-authored-by: Yogesh Tewari <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yogeshtewari yogeshtewari

Labels
priority: p1 High priority. Fix may be included in the next release. Release Next release candidate type: feature request 'Nice-to-have' improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
4 participants
@yogeshtewari @ryanmcdowell @nehanene15 @freedomofnet