Added missing headers to KeenClientFramework target

keenlabs / KeenClient-iOS

Official iOS client for the Keen IO API. Build analytics features directly into your iOS apps.

https://keen.io/docs

MIT License

78 stars 56 forks source link

Added missing headers to KeenClientFramework target #218

Closed capnslipp closed 7 years ago

capnslipp commented 7 years ago

Added headers missing in KeenClientFramework target that are present in KeenClient target, with appropriate access levels:
⁃ KeenLogSink.h (Public, matching KeenClient)
⁃ KeenLogSinkNSLog.h (Project, matching KeenClient)
⁃ KeenLogger.h (Public, because KeenClient.h imports it)

Fixes #216

coveralls commented 7 years ago

Coverage remained the same at 65.882% when pulling 493c2f759d363a393dc8707a7aa9d2a6b7408bc5 on capnslipp:master into 2816c7c07de6c59b44a1b9bf3fe77c66d75ea035 on keenlabs:master.

capnslipp commented 7 years ago

@baumatron Added to KeenClient-Cocoa target too, like you did. Should be good to merge now.

baumatron commented 7 years ago

Great, thanks @capnslipp! I'll work on getting a patch revision shipped with the fix.

coveralls commented 7 years ago

Coverage remained the same at 65.882% when pulling c92114aa15c24a3bc8a846175461241ff027d9fa on capnslipp:master into 2816c7c07de6c59b44a1b9bf3fe77c66d75ea035 on keenlabs:master.