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

Fix issue #10 #11

Merged
merged 4 commits into from
Aug 18, 2015
Merged

Fix issue #10 #11

merged 4 commits into from
Aug 18, 2015

Conversation

serluca
Copy link
Contributor

@serluca serluca commented Aug 3, 2015

Fix the crash when using cocoapods framework, see #10

UIStoryboard *storyboard = [UIStoryboard storyboardWithName:@"LumberjackConsole" bundle:[NSBundle bundleForClass:[self class]]];
self.rootViewController = [storyboard instantiateInitialViewController];
Copy link
Member

Choose a reason for hiding this comment

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

Looks good. Just what happened to the indentation?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, probably we are using different indentation settings

Copy link
Member

Choose a reason for hiding this comment

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

Yes, probably mines are the default ones. Could you just fix them so I can merge it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be ok now, the problem is that it is perfectly indented in my Xcode.
Xcode automatically align the indentation

@rivera-ernesto
Copy link
Member

Seems that your problem is that your indentation uses tabs instead of spaces, so depending on the editor it will always look off (the reason why tabs are discouraged.

@serluca
Copy link
Contributor Author

serluca commented Aug 5, 2015

Is the last commit still not good?
Can you give me your indentation settings? I don't know how to do, it looks perfectly aligned for me

(UIInterfaceOrientationIsPortrait(currentOrientation) && UIInterfaceOrientationIsLandscape(nextOrientation)))
{
frame.size = CGSizeMake(frame.size.height,frame.size.width);
}
Copy link
Member

Choose a reason for hiding this comment

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

Drag your mouse in your indentation code here and you'll notice that it indents with either tab for each level.

Regular Xcode indentation (so this and most iOS/OSX projects) uses four spaces for each level of indentation.

You can take a look here at how to change the setting and some discussion about indentation ;)

http://stackoverflow.com/a/3336065/1049134

Copy link
Member

Choose a reason for hiding this comment

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

Also you can see here how your edits don't look indented in GitHub (and so in many editors that don't represent a tab with four spaces).

@serluca
Copy link
Contributor Author

serluca commented Aug 18, 2015

What do you think about the last commit? Is it aligned?

@rivera-ernesto
Copy link
Member

Yes, looks really good now ;)

Thanks for you time.

rivera-ernesto added a commit that referenced this pull request Aug 18, 2015
@rivera-ernesto rivera-ernesto merged commit 16d2763 into PTEz:master Aug 18, 2015
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