Skip to content

Extract templates to stdout #89

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.

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

Merged
merged 2 commits into from
Dec 21, 2021
Merged

Extract templates to stdout #89

merged 2 commits into from
Dec 21, 2021

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