Skip to content
This repository has been archived by the owner on Jun 7, 2020. It is now read-only.

[CHORE] Update RocketChatViewController to reflect latest fixes #2293

Merged
merged 1 commit into from
Nov 8, 2018

Conversation

rafaelks
Copy link
Contributor

@rafaelks rafaelks commented Nov 8, 2018

@RocketChat/ios

This update includes fixes for invalid indexPaths and small code improvements.

@@ -435,3 +443,4 @@ extension RocketChatViewController {
}
}
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File Line Length Violation: File should contain 400 lines or less: currently contains 446 (file_length)
Trailing Newline Violation: Files should have a single trailing newline. (trailing_newline)

strongSelf.internalData = newData
UIView.performWithoutAnimation {
let changeset = StagedChangeset(source: source, target: target)
collectionView.reload(using: changeset, interrupt: { $0.changeCount > 100 }) { newData, changes in

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument. (multiple_closures_with_trailing_closure)

Copy link
Collaborator

@cardoso cardoso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✌️

@rafaelks rafaelks force-pushed the chore/update_rocketchatviewcontroller_sdk branch from 41f1afc to 2d70a65 Compare November 8, 2018 15:11
@codecov
Copy link

codecov bot commented Nov 8, 2018

Codecov Report

Merging #2293 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2293   +/-   ##
========================================
  Coverage    25.96%   25.96%           
========================================
  Files          432      432           
  Lines        15857    15857           
========================================
  Hits          4118     4118           
  Misses       11739    11739

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 3c1c15a...2d70a65. Read the comment docs.

@rafaelks rafaelks merged commit 1769158 into develop Nov 8, 2018
@rafaelks rafaelks deleted the chore/update_rocketchatviewcontroller_sdk branch November 8, 2018 15:41
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants