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.

Sign up for GitHub

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

Jump to bottom

Cleanup whitespaces #109

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Cleanup whitespaces #109

wants to merge 5 commits into from

Conversation

brodycj
Copy link

@brodycj brodycj commented Oct 3, 2018 •
edited
Loading

  • Fixes for Android:
    • remove trailing whitespace
    • remove extra blank line
    • convert tabs to spaces (and space after an if statement)
  • Fixes for iOS
    • remove trailing whitespace
    • remove extra blank lines

Discovered through review of PR #28 (whitespace cleanup should not have been part of the commit in #28).

@brodycj brodycj force-pushed the cleanup-whitespace branch from 328242e to 53c6a34 Compare October 3, 2018 23:49
@brodycj brodycj changed the title Cleanup whitespace [WIP] Cleanup whitespace Oct 3, 2018
android: convert tabs to spaces
3f675af 
(and space after an if statement)
@brodycj brodycj force-pushed the cleanup-whitespace branch from 482fd4d to 3f675af Compare October 3, 2018 23:57
@brodycj brodycj changed the title [WIP] Cleanup whitespace Cleanup whitespaces Oct 3, 2018
@brodycj brodycj mentioned this pull request Oct 4, 2018
Open
@brodycj brodycj mentioned this pull request Oct 30, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement platform: android platform: ios
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brodycj @janpio