Skip to content

Conversation

@polldo
Copy link
Contributor

@polldo polldo commented Dec 20, 2021

Motivation

Use stdout as primary output outlet of extract commands to allow piping the results for whatever reason
(from @manchoz )

Change description

extract commands are now printing out the template rather than saving it into a file:

  • flags for output file have been removed
  • format output depends on the global flag --format

Additional Notes

Reviewer checklist

  • PR address a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.
  • History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well formatted.

@polldo polldo requested a review from eclipse1985 December 20, 2021 10:12
@polldo polldo force-pushed the polldo/extract-stdout branch from 4522f1d to ad187dd Compare December 20, 2021 15:39
@polldo polldo merged commit 7cfa32a into main Dec 21, 2021
@polldo polldo deleted the polldo/extract-stdout branch December 21, 2021 09:43
polldo pushed a commit that referenced this pull request Sep 2, 2022
Use stdout as primary output outlet of extract commands to allow piping the results for whatever reason
polldo pushed a commit that referenced this pull request Sep 2, 2022
Use stdout as primary output outlet of extract commands to allow piping the results for whatever reason
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants