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

feat: Allow user to specify a format for stdout (#242) #293

Merged
merged 14 commits into from
Aug 3, 2021
Merged

feat: Allow user to specify a format for stdout (#242) #293

merged 14 commits into from
Aug 3, 2021

Conversation

yogeshtewari
Copy link
Contributor

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

@yogeshtewari yogeshtewari changed the title Issue feat 242 feat: Allow user to specify a format for stdout (#242) Jul 30, 2021
@yogeshtewari yogeshtewari self-assigned this Jul 30, 2021
Yogesh Tewari added 8 commits July 30, 2021 12:13
feat: Allow user to specify a format for stdout, added for other test…
76fbea5 
… cases (#242)
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
feat: Allow user to specify a format for stdout, added default parame…
f41759b 
…ter for format (#242)
feat: Allow user to specify a format for stdout, format for text resu…
1259529 
…lt handler (#242)
feat: Allow user to specify a format for stdout, linting fix (#242)
5707b3d
feat: Allow user to specify a format for stdout, files reformatted wi…
f730ac9 
…th black (#242)
Revert "feat: Allow user to specify a format for stdout, files reform…
bb5f91e 
…atted with black (#242)"

This reverts commit f730ac9.
feat: Allow user to specify a format for stdout, files reformatted wi…
4522073 
…th black version 19 (#242)
nehanene15
nehanene15 requested changes Aug 2, 2021
View reviewed changes
setup.py Show resolved Hide resolved
data_validation/cli_tools.py Outdated Show resolved Hide resolved
data_validation/cli_tools.py Show resolved Hide resolved
nehanene15
nehanene15 approved these changes Aug 2, 2021
View reviewed changes
README.md Outdated Show resolved Hide resolved
@yogeshtewari yogeshtewari merged commit f0a9fa1 into develop Aug 3, 2021
@yogeshtewari yogeshtewari deleted the issue_feat_242 branch August 3, 2021 01:50
nehanene15 added a commit that referenced this pull request Aug 4, 2021
@nehanene15
Revert "feat: Allow user to specify a format for stdout (#242) (#293)"
1bbd6b6 
This reverts commit f0a9fa1.
@yogeshtewari yogeshtewari mentioned this pull request Aug 4, 2021
Merged
yogeshtewari added a commit that referenced this pull request 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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nehanene15 nehanene15 nehanene15 approved these changes

Assignees

@yogeshtewari yogeshtewari

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@yogeshtewari @nehanene15