Skip to content

[AIRFLOW-442] SFTPHook - #2465

Closed
sdiazb wants to merge 21 commits into
apache:masterfrom
sdiazb:master
Closed

[AIRFLOW-442] SFTPHook#2465
sdiazb wants to merge 21 commits into
apache:masterfrom
sdiazb:master

Conversation

@sdiazb

@sdiazb sdiazb commented Jul 21, 2017

Copy link
Copy Markdown
Contributor

Dear Airflow maintainers,

Please accept this PR.

JIRA

Description

  • SFTP Hook.

Tests

@codecov-io

codecov-io commented Jul 21, 2017

Copy link
Copy Markdown

Codecov Report

Merging #2465 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2465      +/-   ##
==========================================
+ Coverage   69.39%   69.43%   +0.03%     
==========================================
  Files         146      146              
  Lines       11349    11350       +1     
==========================================
+ Hits         7876     7881       +5     
+ Misses       3473     3469       -4
Impacted Files Coverage Δ
airflow/utils/db.py 84.55% <100%> (+0.12%) ⬆️
airflow/jobs.py 77.12% <0%> (+0.38%) ⬆️

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 aa64f37...79a7202. Read the comment docs.

@bolkedebruin

Copy link
Copy Markdown
Contributor

Hello @sdiazb, if you want use to review this contribution please make sure it is clean and squashed and adheres to the commit guidelines.

@sdiazb

sdiazb commented Jul 24, 2017

Copy link
Copy Markdown
Contributor Author

Thanks @bolkedebruin, I'll close this one and create another following the commit guidelines and squashing the commits.

@sdiazb sdiazb closed this Jul 24, 2017
@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.

4 participants