Skip to content

Conversation

@ashb
Copy link
Member

@ashb ashb commented Mar 25, 2021

These sections only produce output when verbose output is enabled,
otherwise. In all other times they just have extra sections to look at.

This also removes the log lines about redirected output or settings that
can be tweaked when run under GitHub Actions -- it's not relevant there!

Before:

image

(Not these sections are already expanded.)

After:

(Will populate once it has run once)
image

These sections only produce output when verbose output is enabled,
otherwise. In all other times they just have extra sections to look at.

This also removes the log lines about redirected output or settings that
can be tweaked when run under GitHub Actions -- it's not relevant there!
@ashb ashb requested a review from potiuk as a code owner March 25, 2021 13:42
@ashb ashb requested a review from kaxil March 25, 2021 13:48
@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Mar 25, 2021
@github-actions
Copy link

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@ashb ashb merged commit f94cf99 into apache:master Mar 25, 2021
@ashb ashb deleted the remove-empty-sections branch February 14, 2022 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants