Skip to content

Log all breeze output to a file automatically#14470

Merged
potiuk merged 2 commits into
apache:masterfrom
potiuk:log-all-breeze-commands
Feb 26, 2021
Merged

Log all breeze output to a file automatically#14470
potiuk merged 2 commits into
apache:masterfrom
potiuk:log-all-breeze-commands

Conversation

@potiuk

@potiuk potiuk commented Feb 25, 2021

Copy link
Copy Markdown
Member

Fixes: #14390


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

@potiuk

potiuk commented Feb 25, 2021

Copy link
Copy Markdown
Member Author

cc: @ecerulm

Comment thread BREEZE.rst Outdated
Co-authored-by: James Timmins <jameshtimmins@gmail.com>
@github-actions github-actions Bot added the okay to merge It's ok to merge this PR as it does not require more tests label Feb 26, 2021
@github-actions

Copy link
Copy Markdown
Contributor

The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full test matrix is needed and add the 'full tests needed' label. Then you should rebase it to the latest master or amend the last commit of the PR, and push it with --force-with-lease.

@potiuk potiuk merged commit 4a54292 into apache:master Feb 26, 2021
@potiuk potiuk deleted the log-all-breeze-commands branch February 26, 2021 20:49
potiuk added a commit that referenced this pull request Mar 3, 2021
potiuk added a commit that referenced this pull request Mar 3, 2021
potiuk added a commit that referenced this pull request Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools okay to merge It's ok to merge this PR as it does not require more tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

breeze: capture output to a file

3 participants