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

Fix memory layout issues with DDLogLevel #116

Closed jkmassel closed 4 years ago

jkmassel commented 4 years ago

Rather than using int as the type, we use DDLogLevel per https://github.com/CocoaLumberjack/CocoaLumberjack/blob/master/Documentation/DynamicLogLevels.md.

We'll need to merge https://github.com/Automattic/Automattic-Tracks-iOS/pull/117 and rebase this branch before CI will pass for this PR.

jkmassel commented 4 years ago

Tests are now passing following the rebase, and given the 👍 on the code from @koke, I'm confident it's ok to merge this.