Skip to content

Conversation

f2prateek
Copy link
Contributor

We only set this key on tvOS, so let's remove it only on tvOS as well.

Ref LIB-130, closes #671.

We only set this key on tvOS, so let's remove it only on tvOS as well.

Ref LIB-130, closes #671.
@codecov-io
Copy link

Codecov Report

Merging #750 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #750      +/-   ##
==========================================
- Coverage   86.67%   86.67%   -0.01%     
==========================================
  Files          36       36              
  Lines        2582     2581       -1     
  Branches      177      176       -1     
==========================================
- Hits         2238     2237       -1     
  Misses        339      339              
  Partials        5        5

@f2prateek f2prateek merged commit 1478e8f into master Jan 29, 2018
@f2prateek f2prateek deleted the reset-SEGUserIdKey-tvOS branch January 29, 2018 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unnecessary removal of SEGUserIdKey key on different targets than TARGET_OS_TV.
2 participants