Skip to content

Remove redundant parenthesis#12213

Merged
kaxil merged 1 commit intoapache:masterfrom
astronomer:remove-red-parenthesis
Nov 9, 2020
Merged

Remove redundant parenthesis#12213
kaxil merged 1 commit intoapache:masterfrom
astronomer:remove-red-parenthesis

Conversation

@kaxil
Copy link
Copy Markdown
Member

@kaxil kaxil commented Nov 9, 2020

Example:

- ("Here's the list of task instances you are about to clear:"),
+ "Here's the list of task instances you are about to clear:",

^ 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.

@kaxil kaxil requested a review from XD-DENG November 9, 2020 21:27
@boring-cyborg boring-cyborg bot added the area:webserver Webserver related Issues label Nov 9, 2020
@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 Nov 9, 2020
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 9, 2020

The PR should be OK to be merged with just subset of tests as it does not modify Core of Airflow. The committers might merge it or can add a label 'full tests needed' and re-run it to run all tests if they see it is needed!

@kaxil kaxil merged commit a7272f4 into apache:master Nov 9, 2020
@kaxil kaxil deleted the remove-red-parenthesis branch November 9, 2020 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:webserver Webserver related Issues 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.

3 participants