Skip to content

Enabling Source Clear #255

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 6, 2018
Merged

Enabling Source Clear #255

merged 1 commit into from
Jun 6, 2018

Conversation

aliabbasrizvi
Copy link
Contributor

No description provided.

@aliabbasrizvi aliabbasrizvi requested a review from a team June 6, 2018 17:03
@coveralls
Copy link

coveralls commented Jun 6, 2018

Coverage Status

Coverage decreased (-0.4%) to 97.022% when pulling 63c6560 on ali/enable_src_clr into 582a1e2 on master.

Copy link
Contributor

@mikeproeng37 mikeproeng37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@aliabbasrizvi aliabbasrizvi merged commit 390856e into master Jun 6, 2018
@aliabbasrizvi aliabbasrizvi deleted the ali/enable_src_clr branch June 6, 2018 18:12
kellyroach-optimizely added a commit that referenced this pull request Jun 8, 2018
kellyroach-optimizely added a commit that referenced this pull request Jun 8, 2018
* Update OptimizelyiOSDemoApp to Swift 4

* 'pod update' and Localytics version update

* Localytics version update

* Revert "Enabling Source Clear (#255)"

This reverts commit 390856e.

* Xcode 9.4 recommended upgrade to Pods project settings
kellyroach-optimizely added a commit that referenced this pull request Jun 26, 2018
* revert get enabled features sorted (#229)

* coveralls code coverage (#221)

* coveralls added
* enable code coverage

* add style guide (#233)

* data file CDN url changed (#234)

* example to show folio3 how to refactor code (#238)

* example to show folio3 how to refactor code

* write failing test

* fix single quote test

* get record id to attach with event to remove

* fix for TVOS events removal

* Remedying Xcode 9.2 compiler warnings in unit tests (#244)

* use new notification listeners (#243)

* use new notification listeners

* fix TVOS errors and rename notification listener methods that were incorrect

* Kroach/oasis 2519 release20 master (#246)

* Release iOS 2.0.0-beta1 (#230)

* Bump version number to 2.2.0-beta1

* Remove unused Beta and RC XCBuildConfiguration's

* Update workspace *.xcscheme's and *.pbxproj to Xcode 9.2 recommended settings

* Remove static library 'Pods...' from tvOS demo app's Embedded Binaries

* CURRENT_PROJECT_VERSION's can't have version suffixes like '-beta1'

* Additional OptimizelyTVOSDemoApp images required for App Store 'Validation...'

* unexported_symbols.txt update

* Update release.sh

* Rebuild universal frameworks

* CHANGELOG.md update

* Major release.sh overhaul. + Conflate all _VERSION's to single OPTIMIZELY_SDK_VERSION .

* Improved release.sh

* Resolve conflicts between '2.0.x' and 'master'

* Release 2.0.0 GA (#245)

* Bump version number to 2.0.0

* 'Optimizely Bucketing ID' --> '' per reference java-sdk and design doc

* Update CHANGELOG.md

* Add 'Bug Fixes' to CHANGELOG.md

* unexported_symbols.txt update

* Rebuild universal frameworks

* Add 1.5.1 message to CHANGELOG.md

* Merge 2.0.0 GA changes from 2.0.x into master branch.

Summary:
Pull request 2.0.0 GA changes made to 2.0.x branch into the
master branch.  This amounts to cherry-picking commits from
2.0.x and resolving conflicts.

Test Plan:
* sh ./Scripts/build_all.sh PASSED
* 4 Analyze PASSED
* 8+1 iOS test suites PASSED
* 7+1 tvOS test suites PASSED
We seemed to have some issues with tvos test suites last
week, but they PASSED 3x in a row today.

Reviewers: mike.ng, thomas.zurkan

JIRA Issues: OASIS-2519

Differential Revision: https://phabricator.optimizely.com/D19752

* Resolve conflicts (generated-frameworks) (#248)

* fix to send new events to new endpoint and any old events that might … (#249)

* fix to send new events to new endpoint and any old events that might be around to the old events endpoints

* use clientEngine instead of client_engine

* add a couple of tests for sending to new and old endpoints

* Kroach/oasis 2768 upgradev2 (#251)

* Forcing old datafile to include anonymizeIP required by V4 format.

* Add testOlderV2Datafile unit test

* Pre-Release 2.0.2-beta1 (#252) (#253)

* fix to send new events to new endpoint and any old events that might … (#249)

* fix to send new events to new endpoint and any old events that might be around to the old events endpoints

* use clientEngine instead of client_engine

* add a couple of tests for sending to new and old endpoints

* Kroach/oasis 2768 upgradev2 (#251)

* Forcing old datafile to include anonymizeIP required by V4 format.

* Add testOlderV2Datafile unit test

* Bump version number to 2.0.2-beta1

* Rebuild universal frameworks

* Update CHANGELOG.md

* Enabling Source Clear (#255)

* Kroach/oasis 2442 xcode94 (#254)

* Update Project Settings to Xcode 9.4 recommended settings

* Add ', strong, nonnull' to anonymizeIP @Property

* Accept Xcode 9.4's recommended Fix's which add 'self->'

* Eliminate Xcode Aanalyze warning for line in OPTLYFMDBDatabase.m

* Add weakSelf and strongSelf to setupNetworkTimer:

* setupNetworkTimer: nit

* Kroach/oasis 2442 swift (#256)

* Update OptimizelyiOSDemoApp to Swift 4

* 'pod update' and Localytics version update

* Localytics version update

* Revert "Enabling Source Clear (#255)"

This reverts commit 390856e.

* Xcode 9.4 recommended upgrade to Pods project settings

* Kroach/oasis 2978 variables (#257)

* Revive Live Variables API's

* Add 2 V3.json datafiles

* Repair all broken tests.  Make OptimizelyV3Test live variables tests pass.

* Add Live Variables (DEPRECATED) code comments

* Add 2 'Mobile 1.x Live Variables are DEPRECATED' block comments

* Add OPTLYLoggerMessagesLiveVariablesDeprecated log message to deprecated API's

* Add #pragms's to get deprecated warnings to emit in old code using Mobile 1.x Variables

* Managment -> Management

* OPTLYDatafileKeysVariationVariableUsageInstances --> OPTLYDatafileKeysVariationVariables

* re-enabled srcclr on every build (#263)

* Added SourceClear to run in one build per commit (#258)

* added sourceclear to only run in one build per commit

* moved sourceclear.sh into Scripts folder

* fix typo

* it -> is, 'skipping sourceclear'

* Add Release 1.5.2 message to master CHANGELOG.md (#267)

* Fix impression sent from feature experiment variation toggled off. (#270)

* Add environments support to SDK with SDK key initialization (#264)

* iOS Environment refactor to use OPTLYDatafileConfig

Test Plan: all tests pass on iOS and tvOS.

Reviewers: ali, kelly.roach, thomas.zurkan

Differential Revision: https://phabricator.optimizely.com/D19924

* Add unit test for initializing with SDK and fix a typo.

* Undo disabling of MxP.

* projectConfigURLPath: declaration removed

* Improve initWithProjectId:withSDKKey:

* Use BOOL values YES/NO

* Add parentheses and formatting to improve code readability

* Add areNilOrEqual:y: and refactor isEqual: for code clarity

* Improved OPTLYDatafileConfig.m hash method

* NSString+OPTLYCategory.m+.h dropped in favor of adding static method to OPTLYManagerBase.m+.h

* Refer to 6 identical strings by SSOT OPTLY_DATAFILE_URL global

* Changed enum type SqliteValueType to OPTLYSqliteValueType (#271)

* changed enum type SqliteValueType to OPTLYSqliteValueType

* Copyright headers updated

* Suppress deprecated warnings in OptimizelyV3Test

* Correct downloadDatafile:completionHandler: signature in OPTLYDatafileManagerTest.m

* Suppress -Wnonnull warnings in tests for illegitmate args

* Two nil --> [NSData data] in OPTLYDatafileManagerTest.m

* 3x (NSObject *) casts in OPTLYDecisionServiceTest.m

* OPTLYUserProfile --> NSDictionary in 2 places in OPTLYDecisionServiceTest.m

* Bump version number to 2.0.2-beta2

* Update CHANGELOG.md and 2 code comments in OPTLYManagerBase.h

* Improved unexported_symbols.sh

* unexported_symbols.sh comment

* Rebuild universal frameworks
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.

3 participants