Skip to content

[AIRFLOW-1442] Remove extra space from ignore_all_deps generated command#2468

Closed
aoen wants to merge 1 commit into
apache:masterfrom
aoen:ddavydov--fix_ignore_all_deps_extra_space
Closed

[AIRFLOW-1442] Remove extra space from ignore_all_deps generated command#2468
aoen wants to merge 1 commit into
apache:masterfrom
aoen:ddavydov--fix_ignore_all_deps_extra_space

Conversation

@aoen

@aoen aoen commented Jul 21, 2017

Copy link
Copy Markdown
Contributor

My PR addresses the following Airflow JIRA issues and references them in the PR title.

Fix extra whitespace in the ignore_all_deps arg which was causing commands to fail.

@saguziel

@mention-bot

Copy link
Copy Markdown

@aoen, thanks for your PR! By analyzing the history of the files in this pull request, we identified @mistercrunch, @bolkedebruin and @jlowin to be potential reviewers.

@saguziel

Copy link
Copy Markdown
Contributor

lgtm

@bolkedebruin

Copy link
Copy Markdown
Contributor

lol 👍

@asfgit asfgit closed this in aa64f37 Jul 21, 2017
@codecov-io

codecov-io commented Jul 21, 2017

Copy link
Copy Markdown

Codecov Report

Merging #2468 into master will increase coverage by 0.53%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2468      +/-   ##
==========================================
+ Coverage   69.34%   69.87%   +0.53%     
==========================================
  Files         146      146              
  Lines       11331    11764     +433     
==========================================
+ Hits         7857     8220     +363     
- Misses       3474     3544      +70
Impacted Files Coverage Δ
airflow/models.py 87.32% <100%> (+0.04%) ⬆️
airflow/jobs.py 78.48% <0%> (+2.14%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f1f022c...374e97b. Read the comment docs.

@shahar1 shahar1 mentioned this pull request Jul 10, 2026
1 task
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.

5 participants