-
Notifications
You must be signed in to change notification settings - Fork 187
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
Code cleanup, comments, analyzer warnings. #13
Conversation
/** | ||
* Creates a new @c RZUniqueTransition for use with the RZTransition Manager | ||
* | ||
* @param action The action that is to be used in the presentation/dismisall of the View controller |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: dismissal
. And view controller requires no capital letters.
@ZevEisenberg if you want to just make changes using the github inline editor, I am fine with that. It will probably save time for both of us in me going back and updating everything, and you making the comments. |
/** | ||
* The View Controller that is being transitioned from. | ||
*/ | ||
@property(strong, nonatomic) UIViewController *fromViewController; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Space after @property
@Arrouse good call. Will do. |
@Arrouse done. |
Thanks @ZevEisenberg Appreciate you taking the time to look through it. |
@Raizlabs/maintainers-ios I am going to merge this eod. Speak up if you want me to hold off. |
Code cleanup, comments, analyzer warnings.
@Raizlabs/maintainers-ios This is just some cleanup of this repo.
Any input around the content in comments/code style can just be made on this branch, unless a discussion is desired.