Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass view controller provider to PhotosInputView #558

Merged
merged 2 commits into from
Mar 14, 2019

Conversation

wiruzx
Copy link
Contributor

@wiruzx wiruzx commented Mar 13, 2019

No description provided.

@wiruzx wiruzx changed the title Pass view controller provider Pass view controller provider to PhotosInputView Mar 13, 2019
@codecov-io
Copy link

codecov-io commented Mar 13, 2019

Codecov Report

Merging #558 into master will increase coverage by 0.32%.
The diff coverage is 35.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #558      +/-   ##
==========================================
+ Coverage   59.69%   60.02%   +0.32%     
==========================================
  Files          87       87              
  Lines        3960     3945      -15     
==========================================
+ Hits         2364     2368       +4     
+ Misses       1596     1577      -19
Impacted Files Coverage Δ
...dditions/Source/Input/Photos/PhotosInputView.swift 1.73% <0%> (+0.22%) ⬆️
.../Input/Photos/Camera/PhotosInputCameraPicker.swift 100% <100%> (ø) ⬆️
...Controller/BaseChatViewController+Presenters.swift 61.29% <0%> (+3.22%) ⬆️

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 1f0e40d...9c44d4a. Read the comment docs.

@wiruzx wiruzx force-pushed the pass-view-controller-provider branch from ec76013 to 9c44d4a Compare March 13, 2019 20:15
@wiruzx wiruzx merged commit 5995970 into master Mar 14, 2019
@wiruzx wiruzx deleted the pass-view-controller-provider branch March 14, 2019 15:58
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.

2 participants