search

hyperlab / TiMixpanel

Mixpanel integration for Titanium Mobile
22 stars 12 forks source link

Added support for 64-bit #12

Closed timanrebel closed 9 years ago

jonatansberg commented 9 years ago

Sweet! The TiMixpanel.xccheckout-stuff should probably not be in the repo tho.

timanrebel commented 9 years ago

It's part of the TiMixpanel.xcodeproj Xcode file and don't know how to change it. Do you?

jonatansberg commented 9 years ago

Adding xcuserdata/ to the iOS .gitignore-file and removing the files from the repository should fix it.

TiMixpanel.xcodeprojis just a normal bundle, so it's pretty much just a directory with a file extension, and it is treated like any other directory by git.

timanrebel commented 9 years ago

Done!

jonatansberg commented 9 years ago

Great! There are still some files in there that should probably be left out.. A quick search lead me to the following .gitignore for Xcode from https://www.gitignore.io/:

### Xcode ###
build/
*.pbxuser
!default.pbxuser
*.mode1v3
!default.mode1v3
*.mode2v3
!default.mode2v3
*.perspectivev3
!default.perspectivev3
xcuserdata
*.xccheckout
*.moved-aside
DerivedData
*.xcuserstate

If you feel like fixing, then please go ahead! Otherwise I'll just pull this as is, since its slightly outside the scope of this pull request anyway.

And as always, thank you!

timanrebel commented 9 years ago

Let me unstage the files with the next change. I updated the .gitignore with your suggestion

jonatansberg commented 9 years ago

:+1: