Automattic / Automattic-Tracks-iOS

Client library for tracking user events for later analysis
GNU General Public License v2.0
43 stars 12 forks source link

Reorganize the project #124

Closed jkmassel closed 4 years ago

jkmassel commented 4 years ago

Organizes the project file system to match the project file before we make too many more changes. I've broken this down into several smaller commits to group things changes by folder in order to make review easier.

To Test

Future Steps The majority of code doesn't actually belong to the iOS target, but it's there because that one came first. It might make sense to extract the majority of the code to a shared (and sharedTests?) directory. WDYT?

jkmassel commented 4 years ago

I don't think these belong inside the Model folder. TracksContextManager is an important class that should be easily discoverable especially because it's used directly by consumers. Perhaps a top-level Core Data folder would be better.

Agreed – perhaps that could be a future change.

My gut feeling is that there should be a shared "project", not just a directory. But I don't have a lot of experience with how that works so I can't really say if that will work well.

That would probably be a good way to go – CocoaPods doesn't really care about the Xcode project – just the files, so we can organize it sort of however we want.