Skip to content

[AIRFLOW-1193] Add Checkr to company using Airflow#2276

Closed
tongboh wants to merge 1 commit intoapache:masterfrom
tongboh:patch-1
Closed

[AIRFLOW-1193] Add Checkr to company using Airflow#2276
tongboh wants to merge 1 commit intoapache:masterfrom
tongboh:patch-1

Conversation

@tongboh
Copy link

@tongboh tongboh commented May 5, 2017

Dear Airflow maintainers,

Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!

JIRA

Description

  • Here are some details about my PR, including screenshots of any UI changes:
    Add Checkr to companies officially using Airflow.

@mention-bot
Copy link

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

@codecov-io
Copy link

codecov-io commented May 5, 2017

Codecov Report

Merging #2276 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2276      +/-   ##
=========================================
+ Coverage   68.87%   68.9%   +0.02%     
=========================================
  Files         142     142              
  Lines       10969   10969              
=========================================
+ Hits         7555    7558       +3     
+ Misses       3414    3411       -3
Impacted Files Coverage Δ
airflow/models.py 86.96% <0%> (-0.05%) ⬇️
airflow/jobs.py 74.41% <0%> (+0.4%) ⬆️

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 443e6b2...131d8cf. Read the comment docs.

@bolkedebruin
Copy link
Contributor

Please squash your commit

@tongboh
Copy link
Author

tongboh commented May 5, 2017

@bolkedebruin Thanks for jumping on the PR, just squashed commits.

@tongboh tongboh force-pushed the patch-1 branch 2 times, most recently from f88a806 to 2c77002 Compare May 5, 2017 20:18
@tongboh tongboh changed the title Add Checkr to company using Airflow [AIRFLOW-1193] Add Checkr to company using Airflow May 11, 2017
@asfgit asfgit closed this in 7072389 May 12, 2017
alekstorm pushed a commit to alekstorm/incubator-airflow that referenced this pull request Jun 1, 2017
stverhae pushed a commit to stverhae/incubator-airflow that referenced this pull request Jun 6, 2017
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

Comments