Update objc default project template #180
Conversation
|
Works for me. |
| - Main.storyboard | ||
| - Nibs: | ||
| - LaunchScreen.xib | ||
| - Other-Sources: |
There was a problem hiding this comment.
Should other sources be nested under resources? I know it was like that before but it seems like it should be top level.
There was a problem hiding this comment.
I think they're such low churn it makes sense to nest them.
There was a problem hiding this comment.
Yeah, I like that they are stuck out of the way as much as possible.
|
I noticed that you removed 'Constants'. What about adding a group for 'Headers' for things like constants files, category import headers and prefix headers? Other than that, this all looks good. |
|
I removed the Constants group because it usually ended up with a single Constants file in it. But Headers sounds like a better place to move that. I'll add a generic Headers group. |
618a403 to
9138714
Compare
This brings the objc project template in line with the swift template. This is actually a lighter weight template, but I think that makes it a better starting point.
This brings the objc project template in line with the swift template. This is
actually a lighter weight template, but I think that makes it a better
starting point.