Skip to content

Conversation

@lmazuel
Copy link
Member

@lmazuel lmazuel commented Nov 20, 2017

@lmazuel lmazuel requested a review from hvermis November 20, 2017 20:45
@codecov-io
Copy link

codecov-io commented Nov 20, 2017

Codecov Report

Merging #1636 into master will increase coverage by <.01%.
The diff coverage is 57.3%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1636      +/-   ##
==========================================
+ Coverage   56.08%   56.09%   +<.01%     
==========================================
  Files        3904     3993      +89     
  Lines       94289    95284     +995     
==========================================
+ Hits        52885    53449     +564     
- Misses      41404    41835     +431
Impacted Files Coverage Δ
...ry/azure/mgmt/datafactory/models/linked_service.py 100% <ø> (ø) ⬆️
...ctory/azure/mgmt/datafactory/models/copy_source.py 100% <ø> (ø) ⬆️
...ure/mgmt/datafactory/models/dataset_compression.py 85.71% <ø> (ø) ⬆️
...ctory/models/redirect_incompatible_row_settings.py 71.42% <ø> (ø) ⬆️
...mt/datafactory/models/salesforce_linked_service.py 41.66% <ø> (ø) ⬆️
...ctory/azure/mgmt/datafactory/models/hdfs_source.py 55.55% <ø> (ø) ⬆️
...tafactory/azure/mgmt/datafactory/models/dataset.py 100% <ø> (ø) ⬆️
...factory/azure/mgmt/datafactory/models/copy_sink.py 100% <ø> (ø) ⬆️
...tory/azure/mgmt/datafactory/models/http_dataset.py 38.46% <100%> (+2.74%) ⬆️
...tafactory/azure/mgmt/datafactory/models/trigger.py 66.66% <100%> (+6.66%) ⬆️
... and 191 more

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 1898ec4...38b5941. Read the comment docs.

@hvermis
Copy link
Contributor

hvermis commented Nov 20, 2017

Looks good.

@lmazuel
Copy link
Member Author

lmazuel commented Nov 22, 2017

Included in #1640

@lmazuel lmazuel closed this Nov 22, 2017
@lmazuel lmazuel deleted the datafactory_pr2016 branch December 1, 2017 19:45
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