Skip to content

[AIRFLOW-1247] Fix ignore all dependencies argument ignored - #2441

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

[AIRFLOW-1247] Fix ignore all dependencies argument ignored#2441
aoen wants to merge 1 commit into
apache:masterfrom
aoen:ddavydov--fix_ignore_all_deps_cli

Conversation

@aoen

@aoen aoen commented Jul 13, 2017

Copy link
Copy Markdown
Contributor

@saguziel @bolkedebruin

Copy of #2327 without the tests since they seem broken.

JIRA

Tests

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

@codecov-io

codecov-io commented Jul 13, 2017

Copy link
Copy Markdown

Codecov Report

Merging #2441 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2441      +/-   ##
==========================================
- Coverage   69.31%   69.28%   -0.04%     
==========================================
  Files         146      146              
  Lines       11244    11244              
==========================================
- Hits         7794     7790       -4     
- Misses       3450     3454       +4
Impacted Files Coverage Δ
airflow/bin/cli.py 51.86% <ø> (ø) ⬆️
airflow/jobs.py 75.05% <0%> (-0.41%) ⬇️

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 38c86bb...d93d237. Read the comment docs.

@saguziel

Copy link
Copy Markdown
Contributor

lgtm if you confirm working

@asfgit asfgit closed this in e88ecff Jul 13, 2017
ronfung pushed a commit to ronfung/incubator-airflow that referenced this pull request Jul 17, 2017
Fix typo in ignore_all_dependencies argument to fix it.

Closes apache#2441 from aoen/ddavydov--
fix_ignore_all_deps_cli
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.

4 participants